This also cleans up some of the documentation. I'm not sure what happened to that last line.
BUG=webm:1501
Change-Id: Ie0ff0607ebb60d033e7ad40bdc7943c0a6bd6783
The maximum recursion depth is set to 25. If that's not sufficient for
someone's needs, they're welcome to submit a patch that makes it
configurable.
This fixes a stack overflow found by AutoFuzz. The stack overflow was
caused by recursing too much. I've included the fuzzer's test cases in
the fuzzing/corpus directory. I've also created two (one for SimpleTag
and ChapterAtom each) additional test files, each comprised of 10,000
recursively nested elements. Manually running the fuzzer (with ASan and
UBSan) shows all inputs are now handled correctly, and no stack
overflows occur.
Change-Id: I8514259fd0788e71a58e3ccce2a0fb8a3523acfc
Using a webm/ prefix requires clients to add the webm_parser/include
path to their header search paths. This is inconvenient and an
unnecessary restriction.
Change-Id: If73a4433ad42f326a544766beca5f05710f52243
Mass warning clean up. Mainly:
- Explicit casts of numeric literals to avoid signed/unsigned compare
warnings.
- Commenting out of unused function arg names.
Change-Id: I0e70393a5743ae984035d43712c724d4ccd12f9d
