This is an archive of the discontinued LLVM Phabricator instance.

Fix null dereference in yaml::Document::skip
ClosedPublic

Authored by thomasfinch on Nov 7 2019, 3:08 PM.

Details

Summary

The attached test case replicates a null dereference crash in yaml::Document::skip(). This was fixed by adding a check and early return in the method.

Diff Detail

Event Timeline

thomasfinch created this revision.Nov 7 2019, 3:08 PM
hintonda accepted this revision.Nov 9 2019, 11:02 AM

LGTM, but I'll wait a few days to land it for you to give @Bigcheese a chance to comment.

This revision is now accepted and ready to land.Nov 9 2019, 11:02 AM
This revision was automatically updated to reflect the committed changes.