search

openpreserve / jhove

File validation and characterisation.
http://jhove.openpreservation.org
Other
171 stars 79 forks source link

Unhandled exception when Size is not an integer #744

Closed samalloing closed 1 year ago

samalloing commented 2 years ago

Example file: https://github.com/qpdf/qpdf/blob/main/qpdf/qtest/qpdf/bad10.pdf

samalloing commented 2 years ago

Sorry @carlwilson I mixed two pull requests again.

codecov[bot] commented 2 years ago

Codecov Report

Base: 46.33% // Head: 46.33% // No change to project coverage :thumbsup:

Coverage data is based on head (ba56767) compared to base (eadc199). Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## integration #744 +/- ## ============================================== Coverage 46.33% 46.33% Complexity 1054 1054 ============================================== Files 57 57 Lines 9079 9079 Branches 1612 1612 ============================================== Hits 4207 4207 Misses 4333 4333 Partials 539 539 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openpreserve). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openpreserve)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

carlwilson commented 2 years ago

I'm going to push out a (final?) RC this week with a small fix for internationalisation defaults. I'd delayed the release to see if anything else came along this week and this seems to fit the bill. I'll take a look and merge if I can.