search

IanYates / pdfium

Automatically exported from code.google.com/p/pdfium
0 stars 0 forks source link

[PATCH] Fix resource leaks in fpdfsdk/src/javascript/Document.cpp #12

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Please review the attached patch. It fixes two potential resource leaks in 
fpdfsdk/src/javascript/Document.cpp.

Best regards and many thanks

Martin Ettl

Original issue reported on code.google.com by ettl.mar...@gmail.com on 13 Jun 2014 at 11:55

Attachments:

GoogleCodeExporter commented 9 years ago 
Thank you for the patch. For reviews, we follow Chromium's model: 
http://dev.chromium.org/developers/contributing-code#TOC-Create-a-change-list-CL
-

Original comment by jam@chromium.org on 13 Jun 2014 at 5:50

GoogleCodeExporter commented 9 years ago 
Martin, have you tried to upload the patch? Once that's done, we can review and 
help you commit it. Thanks

Original comment by bo...@foxitsoftware.com on 18 Jul 2014 at 7:37

GoogleCodeExporter commented 9 years ago 
https://codereview.chromium.org/903893002 removed the code in question a few 
months ago.

Original comment by thestig@chromium.org on 8 May 2015 at 9:34