db2a8b
--- poppler/glib/poppler-document.cc
db2a8b
+++ poppler/glib/poppler-document.cc
780150
@@ -3405,6 +3405,7 @@ PopplerFormField *poppler_document_get_f
780150
     unsigned fieldNum;
780150
     FormPageWidgets *widgets;
780150
     FormWidget *field;
780150
+    PopplerFormField *formField;
780150
 
780150
     FormWidget::decodeID(id, &pageNum, &fieldNum);
780150
 
780150
@@ -3417,8 +3418,14 @@ PopplerFormField *poppler_document_get_f
780150
         return nullptr;
780150
 
780150
     field = widgets->getWidget(fieldNum);
780150
-    if (field)
780150
-        return _poppler_form_field_new(document, field);
780150
+    if (field) {
780150
+        formField = _poppler_form_field_new(document, field);
780150
+        delete widgets;
db2a8b
+
780150
+        return formField;
780150
+    }
db2a8b
+
780150
+    delete widgets;
db2a8b
 
780150
     return nullptr;
db2a8b
 }
db2a8b
--- poppler/poppler/CairoOutputDev.cc
db2a8b
+++ poppler/poppler/CairoOutputDev.cc
780150
@@ -2921,8 +2921,10 @@ void CairoOutputDev::setMimeData(GfxStat
780150
 
780150
     // colorspace in stream dict may be different from colorspace in jpx
780150
     // data
780150
-    if (strKind == strJPX && colorSpace)
780150
+    if (strKind == strJPX && colorSpace) {
780150
+        delete colorSpace;
780150
         return;
780150
+    }
780150
 
780150
     // only embed mime data for gray, rgb, and cmyk colorspaces.
780150
     if (colorSpace) {
db2a8b
--- poppler/poppler/TextOutputDev.cc
db2a8b
+++ poppler/poppler/TextOutputDev.cc
780150
@@ -1619,7 +1619,6 @@ TextBlock::~TextBlock()
780150
 
780150
 void TextBlock::addWord(TextWord *word)
780150
 {
780150
-    pool->addWord(word);
780150
     if (xMin > xMax) {
780150
         xMin = word->xMin;
780150
         xMax = word->xMax;
780150
@@ -1639,6 +1638,7 @@ void TextBlock::addWord(TextWord *word)
780150
             yMax = word->yMax;
780150
         }
db2a8b
     }
780150
+    pool->addWord(word);
db2a8b
 }
db2a8b
 
780150
 void TextBlock::coalesce(const UnicodeMap *uMap, double fixedPitch)
780150
@@ -3064,11 +3064,13 @@ void TextPage::coalesce(bool physLayout,
780150
             word0 = pool->getPool(startBaseIdx);
780150
             pool->setPool(startBaseIdx, word0->next);
780150
             word0->next = nullptr;
780150
-            blk = new TextBlock(this, rot);
780150
-            blk->addWord(word0);
db2a8b
 
780150
             fontSize = word0->fontSize;
780150
             minBase = maxBase = word0->base;
db2a8b
+
780150
+            blk = new TextBlock(this, rot);
780150
+            blk->addWord(word0);
db2a8b
+
780150
             colSpace1 = minColSpacing1 * fontSize;
780150
             colSpace2 = minColSpacing2 * fontSize;
780150
             lineSpace = maxLineSpacingDelta * fontSize;
780150
@@ -3095,9 +3097,9 @@ void TextPage::coalesce(bool physLayout,
780150
                             }
780150
                             word1 = word1->next;
780150
                             word2->next = nullptr;
780150
+                            newMinBase = word2->base;
780150
                             blk->addWord(word2);
780150
                             found = true;
780150
-                            newMinBase = word2->base;
780150
                         } else {
780150
                             word0 = word1;
780150
                             word1 = word1->next;
780150
@@ -3123,9 +3125,9 @@ void TextPage::coalesce(bool physLayout,
780150
                             }
780150
                             word1 = word1->next;
780150
                             word2->next = nullptr;
780150
+                            newMaxBase = word2->base;
780150
                             blk->addWord(word2);
780150
                             found = true;
780150
-                            newMaxBase = word2->base;
780150
                         } else {
780150
                             word0 = word1;
780150
                             word1 = word1->next;
780150
@@ -3198,12 +3200,12 @@ void TextPage::coalesce(bool physLayout,
780150
                                 }
780150
                                 word1 = word1->next;
780150
                                 word2->next = nullptr;
780150
-                                blk->addWord(word2);
780150
                                 if (word2->base < minBase) {
780150
                                     minBase = word2->base;
780150
                                 } else if (word2->base > maxBase) {
780150
                                     maxBase = word2->base;
780150
                                 }
780150
+                                blk->addWord(word2);
780150
                                 found = true;
780150
                                 break;
780150
                             } else {
780150
@@ -3246,12 +3248,12 @@ void TextPage::coalesce(bool physLayout,
780150
                                 }
780150
                                 word1 = word1->next;
780150
                                 word2->next = nullptr;
780150
-                                blk->addWord(word2);
780150
                                 if (word2->base < minBase) {
780150
                                     minBase = word2->base;
780150
                                 } else if (word2->base > maxBase) {
780150
                                     maxBase = word2->base;
780150
                                 }
780150
+                                blk->addWord(word2);
780150
                                 found = true;
780150
                                 break;
780150
                             } else {
db2a8b
--- poppler/poppler/XRef.cc
db2a8b
+++ poppler/poppler/XRef.cc
780150
@@ -402,6 +402,7 @@ int XRef::reserve(int newSize)
780150
 
780150
         void *p = greallocn_checkoverflow(entries, realNewSize, sizeof(XRefEntry));
780150
         if (p == nullptr) {
780150
+            entries = nullptr;
780150
             return 0;
780150
         }
780150
 
780150
@@ -835,7 +836,6 @@ bool XRef::constructXRef(bool *wasRecons
780150
     int offset = 0;
780150
 
780150
     resize(0); // free entries properly
780150
-    gfree(entries);
780150
     capacity = 0;
780150
     size = 0;
780150
     entries = nullptr;
db2a8b
--- poppler/test/pdf-inspector.cc
db2a8b
+++ poppler/test/pdf-inspector.cc
780150
@@ -43,6 +43,7 @@ class PdfInspector
780150
 {
db2a8b
 public:
780150
     PdfInspector();
780150
+    ~PdfInspector();
db2a8b
 
780150
     void set_file_name(const char *file_name);
780150
     void load(const char *file_name);
780150
@@ -108,6 +109,11 @@ PdfInspector::PdfInspector()
780150
     load(nullptr);
db2a8b
 }
780150
 
db2a8b
+PdfInspector::~PdfInspector(void)
db2a8b
+{
780150
+    delete output;
db2a8b
+}
780150
+
780150
 void PdfInspector::set_file_name(const char *file_name)
780150
 {
780150
     GtkWidget *widget;
db2a8b
--- poppler/utils/HtmlOutputDev.cc
db2a8b
+++ poppler/utils/HtmlOutputDev.cc
780150
@@ -1337,6 +1337,7 @@ void HtmlOutputDev::drawPngImage(GfxStat
780150
     // TODO can we calculate the resolution of the image?
780150
     if (!writer->init(f1, width, height, 72, 72)) {
780150
         error(errInternal, -1, "Can't init PNG for image '{0:t}'", fName);
db2a8b
+        delete fName;
db2a8b
         delete writer;
db2a8b
         fclose(f1);
780150
         return;
db2a8b
--- poppler/utils/pdftotext.cc
db2a8b
+++ poppler/utils/pdftotext.cc
780150
@@ -329,6 +329,7 @@ int main(int argc, char *argv[])
780150
             fputs("
\n", f);
780150
             if (f != stdout) {
780150
                 fclose(f);
780150
+                f = nullptr;
780150
             }
780150
         }
db2a8b
     }
780150
@@ -348,8 +349,9 @@ int main(int argc, char *argv[])
780150
                 printWordBBox(f, doc, textOut, firstPage, lastPage);
780150
             }
780150
         }
780150
-        if (f != stdout) {
780150
+        if (f != stdout && f != nullptr) {
780150
             fclose(f);
780150
+            f = nullptr;
780150
         }
780150
     } else {
780150
         textOut = new TextOutputDev(textFileName->c_str(), physLayout, fixedPitch, rawOrder, htmlMeta, discardDiag);
780150
@@ -390,7 +392,7 @@ int main(int argc, char *argv[])
780150
             fputs("\n", f);
780150
         fputs("</body>\n", f);
780150
         fputs("</html>\n", f);
780150
-        if (f != stdout) {
780150
+        if (f != stdout && f != nullptr) {
780150
             fclose(f);
780150
         }
db2a8b
     }