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

List:       kde-commits
Subject:    kdesupport/akode/lib
From:       Hasso Tepper <hasso () kde ! org>
Date:       2007-10-17 13:17:22
Message-ID: 1192627042.226608.11658.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 726298 by hasso:

Don't leak file descriptors if fstat(2) fails for some reason.


 M  +8 -2      localfile.cpp  
 M  +4 -1      mmapfile.cpp  


--- trunk/kdesupport/akode/lib/localfile.cpp #726297:726298
@@ -58,7 +58,10 @@
 
     struct stat stat;
     _fd = ::open(filename, O_RDONLY);
-    if (fstat(_fd, &stat) < 0) return false;
+    if (fstat(_fd, &stat) < 0) {
+        ::close(_fd);
+        return false;
+    }
     len = stat.st_size;
 
     m_readable = true;
@@ -74,7 +77,10 @@
 
     struct stat stat;
     _fd = ::open(filename, O_RDWR);
-    if (fstat(_fd, &stat) < 0) return false;
+    if (fstat(_fd, &stat) < 0) {
+        ::close(_fd);
+        return false;
+    }
     len = stat.st_size;
 
     m_readable = true;
--- trunk/kdesupport/akode/lib/mmapfile.cpp #726297:726298
@@ -58,7 +58,10 @@
     struct stat stat;
 
     fd = ::open(filename, O_RDONLY);
-    if (fstat(fd, &stat) < 0) return false;
+    if (fstat(fd, &stat) < 0) {
+        ::close(fd);
+        return false;
+    }
     len = stat.st_size;
     pos = 0;
 
[prev in list] [next in list] [prev in thread] [next in thread]