This is an archive of the discontinued LLVM Phabricator instance.

Add crash-triggering test case and more null checks in YAML parser
AbandonedPublic

Authored by thomasfinch on Sep 5 2019, 3:40 PM.

Details

Reviewers
None
Summary

Fix crashes when parsing flow tokens outside of a mapping or sequence node

Fix assert when reading a directive then a mapping with no value

Fix infinite loop in unescapeDoubleQuoted for invalid escapes

Make Scanner::consume unexpected characters error nonfatal

Fix ASan-caught use-after-poison with YAML parser SimpleKeys

Add test for unknown directives

Diff Detail

Event Timeline

thomasfinch created this revision.Sep 5 2019, 3:40 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 5 2019, 3:40 PM
thomasfinch abandoned this revision.Sep 5 2019, 3:41 PM

Accidental push...