[PATCH] coverity#982282: Resource leak in object

Julien Nabet (via Code Review) gerrit at gerrit.libreoffice.org
Sat Mar 2 05:48:21 PST 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/2511

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/11/2511/1

coverity#982282: Resource leak in object

Change-Id: Ia7d936c90d7afd4ffa4e22a86289aba47956f576
---
M sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.cxx
M sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.hxx
2 files changed, 5 insertions(+), 0 deletions(-)



diff --git a/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.cxx b/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.cxx
index 15182c2..b556486 100644
--- a/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.cxx
+++ b/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.cxx
@@ -486,6 +486,10 @@
     m_pUtf8Map( new UnicodeMap((char*)"UTF-8", gTrue, &mapUTF8) )
 {
 }
+PDFOutDev::~PDFOutDev()
+{
+    delete m_pUtf8Map;
+}
 
 void PDFOutDev::startPage(int /*pageNum*/, GfxState* state)
 {
diff --git a/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.hxx b/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.hxx
index e5e48fc..c479624 100644
--- a/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.hxx
+++ b/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.hxx
@@ -152,6 +152,7 @@
 
     public:
         explicit PDFOutDev( PDFDoc* pDoc );
+        ~PDFOutDev();
 
         //----- get info about output device
 

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

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



More information about the LibreOffice mailing list