-
Notifications
You must be signed in to change notification settings - Fork 10.3k
Crashes/timeouts on freshly fuzzed files #14305
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Unrelated sidenote...omg, github accepts or rejects files based solely on their file suffix, and it doesn't recognize tgz?! Ommm... So maybe, I'll just throw a .pdf on the end of the tgz file that github didn't like... 🤮 |
All those documents seem to "work" more-or-less about as good as can be expected, when testing with an up-to-date PDF.js version, given that we're dealing with various kinds of corrupt PDF documents. Let's close this as a duplicate of #14303 for now, since that one likely covers any remaining problems already. |
I've rebuilt from |
Adding them to the open issue, i.e. 14303, seems fine assuming they're indeed "new" problems. I've not had time to go all the remaining ones in 14303 in detail, but I believe that there's basically three categories: Bugs already tracked elsewhere, documents which work fine in the viewer (note that Node.js performs worse e.g. on account of no Worker-support), and documents which are so ridiculously huge that it doesn't make much sense to worry about them. |
On #14303, I shared unmodified files from our bug tracker corpus that appeared to cause some problems for pdf.js. On this issue, I'll attach files that are freshly fuzzed using files from that corpus as seeds. I avoided files used in #14303 as seeds for these runs
I haven't done root cause analysis, and I apologize if these are all just triggering the same underlying bug (or that fixed in #14304). The files are named for their seed file and then the sha-256 of the fuzzed version.
new-nasties.zip
The text was updated successfully, but these errors were encountered: