cage1016 / pdfium

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

Crash in PDF_DataDecode (core/src/fpdfapi/fpdf_parser/fpdf_parser_decode.cpp) #3

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
The line
ParamList.Add(pParams->GetDict()); 
may crash if pParams is NULL. This could happen, if pDict parameter has Filter 
property of type PDFOBJ_NAME but doesn't have DecodeParms property.

Original issue reported on code.google.com by halya...@chromium.org on 24 May 2014 at 1:02

GoogleCodeExporter commented 9 years ago
if pParams is NULL, GetDict in fpdf_parser_objects.cpp:146 can check this 
situation and return NULL.

Original comment by bo...@foxitsoftware.com on 25 May 2014 at 5:44

GoogleCodeExporter commented 9 years ago
We shouldn't do that
https://code.google.com/p/pdfium/issues/detail?id=4

Original comment by halya...@chromium.org on 25 May 2014 at 8:12

GoogleCodeExporter commented 9 years ago

Original comment by bo...@foxitsoftware.com on 2 Jun 2014 at 11:05

GoogleCodeExporter commented 9 years ago

Original comment by bo...@foxitsoftware.com on 30 Jun 2014 at 4:45

GoogleCodeExporter commented 9 years ago
Fixed in 
https://pdfium.googlesource.com/pdfium/+/287e11a213d3197ac3e321acf294d903b271c26
9

Original comment by bo...@foxitsoftware.com on 30 Jun 2014 at 4:53