Browse Source

LibCore: Do not leak FILE pointer in Group::add_group()

By using a ScopeGuard we make sure that we always close the FILE, also
on early returns.
Kenneth Myhra 3 năm trước cách đây
mục cha
commit
1fb8408aa2
1 tập tin đã thay đổi với 5 bổ sung2 xóa
  1. 5 2
      Userland/Libraries/LibCore/Group.cpp

+ 5 - 2
Userland/Libraries/LibCore/Group.cpp

@@ -5,6 +5,7 @@
  */
 
 #include <AK/CharacterTypes.h>
+#include <AK/ScopeGuard.h>
 #include <LibCore/Group.h>
 #include <LibCore/System.h>
 
@@ -48,11 +49,13 @@ ErrorOr<void> Group::add_group(Group& group)
     if (!file)
         return Error::from_errno(errno);
 
+    ScopeGuard file_guard { [&] {
+        fclose(file);
+    } };
+
     if (putgrent(&gr, file) < 0)
         return Error::from_errno(errno);
 
-    fclose(file);
-
     return {};
 }
 #endif