[PATCH] coverity#982973: Resource leak

Julien Nabet (via Code Review) gerrit at gerrit.libreoffice.org
Tue May 7 16:10:35 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3821

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/21/3821/1

coverity#982973: Resource leak

Change-Id: I1ea51b29c3ad05016f42c215b71877cd8c3f92f4
---
M extensions/source/nsplugin/source/so_env.cxx
1 file changed, 1 insertion(+), 0 deletions(-)



diff --git a/extensions/source/nsplugin/source/so_env.cxx b/extensions/source/nsplugin/source/so_env.cxx
index 0e8b5f1..b275ad4 100644
--- a/extensions/source/nsplugin/source/so_env.cxx
+++ b/extensions/source/nsplugin/source/so_env.cxx
@@ -132,6 +132,7 @@
         fprintf (stderr, "Can't open myself '%s'\n", dlerror());
         return 1;
     }
+    dlclose(dl_handle);
 
     Dl_info dl_info = { 0,0,0,0 };
     if(!dladdr((void *)nspluginOOoModuleHook, &dl_info))

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1ea51b29c3ad05016f42c215b71877cd8c3f92f4
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Julien Nabet <serval2412 at yahoo.fr>



More information about the LibreOffice mailing list