Fix pdfium-i#236: memory leaks from RenderPdf
authorQin Zhao <zhaoqin@google.com>
Tue, 20 Oct 2015 22:31:06 +0000 (18:31 -0400)
committerQin Zhao <zhaoqin@google.com>
Tue, 20 Oct 2015 22:31:06 +0000 (18:31 -0400)
- add FPDFAvail_Destroy(pdf_avail) on the early return path in RenderPdf

TBR=thestig@chromium.org
BUG=pdfium:236

Review URL: https://codereview.chromium.org/1410333007 .

(cherry picked from commit 0223be82e0afcfee7c501ad2754230353bff79ff)

Review URL: https://codereview.chromium.org/1414913003 .

samples/pdfium_test.cc

index 19b6b46..c9d84af 100644 (file)
@@ -390,6 +390,7 @@ void RenderPdf(const std::string& name, const char* pBuf, size_t len,
 
   if (!doc) {
     fprintf(stderr, "Load pdf docs unsuccessful.\n");
+    FPDFAvail_Destroy(pdf_avail);
     return;
   }