[PATCH libreoffice-4-0] Fix Resource leak: ddf

Julien Nabet (via Code Review) gerrit at gerrit.libreoffice.org
Sun Feb 10 00:17:52 PST 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/2080

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/80/2080/1

Fix Resource leak: ddf

Change-Id: I4361b4f4670a0437c8220d2e7f92f2ffbe0cc479
Reviewed-on: https://gerrit.libreoffice.org/2057
Tested-by: LibreOffice gerrit bot <gerrit at libreoffice.org>
Reviewed-by: Norbert Thiebaud <nthiebaud at gmail.com>
---
M setup_native/source/win32/wintools/makecab/parseddf.c
1 file changed, 2 insertions(+), 0 deletions(-)



diff --git a/setup_native/source/win32/wintools/makecab/parseddf.c b/setup_native/source/win32/wintools/makecab/parseddf.c
index cbd9914..cf081e3 100644
--- a/setup_native/source/win32/wintools/makecab/parseddf.c
+++ b/setup_native/source/win32/wintools/makecab/parseddf.c
@@ -378,6 +378,8 @@
     ccab->cbFolderThresh = ccab->cb;
     ccab->iDisk = 0;
 
+    fclose(ddf);
+
     return DDF_OK;
 }
 

-- 
To view, visit https://gerrit.libreoffice.org/2080
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4361b4f4670a0437c8220d2e7f92f2ffbe0cc479
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Julien Nabet <serval2412 at yahoo.fr>
Gerrit-Reviewer: LibreOffice gerrit bot <gerrit at libreoffice.org>
Gerrit-Reviewer: Norbert Thiebaud <nthiebaud at gmail.com>


More information about the LibreOffice mailing list