[PATCH] coverity#982283: Resource leak in object

Julien Nabet (via Code Review) gerrit at gerrit.libreoffice.org
Fri Mar 1 12:30:21 PST 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/2502

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/02/2502/1

coverity#982283: Resource leak in object

Change-Id: I55fb144adf9c823226dbed31b6ccc92567a7f8bc
---
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/2502
To unsubscribe, visit https://gerrit.libreoffice.org/settings

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



More information about the LibreOffice mailing list