make dump

quick hack to dump /DCTDecode as jpeg files

Merge branch 'fix-assertion-a-used-failed' into 'master' Fix segfault on dictionaries with odd lengths See merge request pesco/pdf!16

Merge branch 'fix-aux-xrefs-segfault' into 'master' Fix segfault when `decode_stream` fails in xrefs See merge request pesco/pdf!17

Fix segfault on dictionaries with odd lengths It’s probably a bug that our dictionary parser is inserting a key-value “pair” into our dictionary structure which just has a key but no value, but the proximal cause of the crash was that `dictentry` is reading off the end of the key-value pair and getting a null pointer. This fixes the bug revealed by the instigator in input file assertion-a-used-failed.

Merge branch 'fix-decode-assert-fail' into 'master' Report incorrect /Filter type with decode failure See merge request pesco/pdf!18

Fix segfault when `decode_stream` fails in xrefs In instigator-crashes/aux-xrefs-segfault an invalid flate-encoded stream was producing this behavior: inflate: invalid distance too far back (-3) parse error in stream (XRef) ../instigator-crashes/aux-xrefs-segfault: error parsing xref section at position 249939 (0x3d053) Program received signal SIGSEGV, Segmentation fault. 0x000055555555d91f in lookup_xref (aux=0x7fffffffdf60, nr=4, gen=0) at pdf.c:1249 1249 HCountedArray *subs = H_INDEX_SEQ(aux->xrefs[i], 0); What was happening was that `act_ks_value`, indirectly invoked by `parse_xrefs`, invoked `decode_stream`, which produced the "inflate:" message and returned NULL; so `act_ks_value` produced the "parse error in stream" message and returned an HParseResult of that NULL pointer. Higher up the stack `act_xrstm` packs this NULL pointer into element 0 of a new `h_sequence`. `parse_xrefs` was happily storing this `h_sequence` into `aux->xrefs[0]`, then blithely continuing to the next loop iteration, at which point it would report "error parsing xref section" and return back to main(). However, this did not abort parsing the file! main() was continuing on to attempt to parse the PDF file as a whole, but the first time the resulting parse tried to `lookup_xref`, that lookup would attempt to iterate over the xrefs section in the file, checking to see if the xref number belonged to any of them. The line of code above then segfaulted while attempting to assert that the NULL was actually a valid `h_sequence` pointer. So this patch simply prevents `parse_xrefs` from treating the failed xrefs section as valid. The result is that, as before, the parse exits shortly because it can't follow any xrefs — but now without segfaulting! inflate: invalid distance too far back (-3) parse error in stream (XRef) ../instigator-crashes/aux-xrefs-segfault: error parsing xref section at position 255242 (0x3e50a) VIOLATION[1]@433 (0x1b1): Missing endobj token (severity=1) ../instigator-crashes/aux-xrefs-segfault: no parse VIOLATION[1]@433 (0x1b1): Missing endobj token (severity=1) ../instigator-crashes/aux-xrefs-segfault: error after position 433 (0x1b1) [Inferior 1 (process 626584) exited with code 01]

Fix overflow in act_rldstring

Merge branch 'xentrac/pdf-fix-digit-pair-assert'

Fix typo in comment

Merge branch 'denleylam/pdf-master'

Report incorrect /Filter type with decode failure Previously, when the instigator produced a PDF file with a stream with `<</Filter 718>>` in its stream dictionary, pdf was failing by aborting with an assert failure. An assert failure is not the right way to report that the program’s input is invalid. This change simply returns NULL from `decode_stream` in this case.

Fix erroneous assert that never worked This bug was only triggered when a PDF stream used AsciiHexDecode, which is very unusual, but it would then always be triggered if the stream contained two or more hex digits.

Fix uninitialized variable for lzwspec

initialize result variables in act_hdigitpair()

improve overflow check in act_numb()

Fix overflow issue

Merge branch 'master' into 2020-06-27_RELEASE

Revert "Execute tests for should_fail directory" This reverts commit fa8c763a8ef8d531ce5876e9e44ec11cc9390f58.

better detect missing newline before 'endstream'

Execute tests for should_fail directory

detect missing linefeed after 'stream' keyword

Move test that's expected to fail

check for integer overflow

ignore but log leading junk bytes