diff options
author | Jay Berkenbilt <ejb@ql.org> | 2019-01-04 17:50:02 +0100 |
---|---|---|
committer | Jay Berkenbilt <ejb@ql.org> | 2019-01-04 17:55:42 +0100 |
commit | 837dcf8fc2546a80f205a0c4c53e5a1545c53a84 (patch) | |
tree | 8c21738d5bb958faadb36ffdaff3f4dbc82cbb35 /TODO | |
parent | a01359189b32c60c2d55b039f7aefd6c3ce0ebde (diff) | |
download | qpdf-837dcf8fc2546a80f205a0c4c53e5a1545c53a84.tar.zst |
Don't call assert while checking linearization data (fixes #209, #231)
Instead of calling assert for problems found during checking
linearization data, throw an exception which is later caught and
issued as an error. Ideally we would handle errors more robustly, but
this is still a significant improvement.
Diffstat (limited to 'TODO')
-rw-r--r-- | TODO | 7 |
1 files changed, 6 insertions, 1 deletions
@@ -391,4 +391,9 @@ I find it useful to make reference to them in this list hint stream contents. Consider adding on option to provide a human-readable dump of linearization hint tables. This should include improving the 'overflow reading bit stream' message as - reported in issue #2. + reported in issue #2. There are multiple calls to stopOnError in + the linearization checking code. Ideally, these should not + terminate checking. It would require re-acquiring an understanding + of all that code to make the checks more robust. In particular, + it's hard to look at the code and quickly determine what is a true + logic error and what could happen because of malformed user input. |