Add fuzzers for internal IndexedDB parsers |
|||||||||
Issue descriptionlibfuzzer-based fuzzers are ridiculously simple to author. We should have them for our various internal string parsers since we're merely human. * blink's IDBKeyPathParser - although the parser is now just a string split it used to be more complex and the isIdentifier() logic is not completely trivial * chromium's leveldb key coding scheme functions: DecodeXXX(), CompareXXX(), ExtractEncodedIDBKey() - anything that takes a StringPiece (The structured clone mechanism is getting its own fuzzer. I couldn't think of any others but we might have them...)
,
Oct 11 2016
I worked on a local fuzzer to test: 1. leveldb::port::Snappy_Compress 2. leveldb::port::Snappy_GetUncompressedLength 3. leveldb::DescriptorFileName A little cleanup and I can land these too.
,
Oct 11 2016
CL showing a fuzzer (plus the implementation, so it's a bit big): https://codereview.chromium.org/1362963003/ Another CL that just adds a fuzzer (the one that inspired me): https://codereview.chromium.org/2402503002/
,
Oct 12 2017
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 20 2017
*snooze*
,
Oct 20 2017
,
Oct 22
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 30
,
Oct 30
,
Nov 9
jsbell@: Is this issue available now?
,
Nov 9
Yes, available! I would focus on the key parsers in content/browser/indexed_db/indexed_db_leveldb_coding.h
,
Nov 28
,
Nov 28
Issue 900468 has been merged into this issue.
,
Nov 28
Assigning to cmumford@ for now, because he mentioned that he'd clean up some previously written fuzzers and upload them. |
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by jsb...@chromium.org
, Oct 11 2016