[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [libqapt] src: Don't leak the file descriptor.
From:       Jonathan Thomas <echidnaman () kubuntu ! org>
Date:       2013-06-18 14:17:47
Message-ID: 20130618141747.0809DA605A () git ! kde ! org
[Download RAW message or body]

Git commit a5775df9627a3e26fb55093ae99f0ff4cc02ba23 by Jonathan Thomas.
Committed on 18/06/2013 at 16:17.
Pushed by jmthomas into branch 'master'.

Don't leak the file descriptor.

M  +2    -0    src/backend.cpp

http://commits.kde.org/libqapt/a5775df9627a3e26fb55093ae99f0ff4cc02ba23

diff --git a/src/backend.cpp b/src/backend.cpp
index e0f4312..b492d06 100644
--- a/src/backend.cpp
+++ b/src/backend.cpp
@@ -1377,6 +1377,7 @@ bool Backend::setPackagePinned(Package *package, bool=
 pin)
 =

             pkgTagFile tagFile(&Fd);
             if (_error->PendingError()) {
+                fclose(out);
                 return false;
             }
 =

@@ -1398,6 +1399,7 @@ bool Backend::setPackagePinned(Package *package, bool=
 pin)
                     fprintf(out, "\n");
                 }
             }
+            fclose(out);
 =

             if (!tempFile.open()) {
                 return false;
[prev in list] [next in list] [prev in thread] [next in thread]