This is an archive of the discontinued LLVM Phabricator instance.

sanitizer_common: fix up onprint.cpp test
ClosedPublic

Authored by dvyukov on Oct 27 2021, 9:03 AM.

Details

Summary

Commit D112602 ("sanitizer_common: tighten on_print hook test")
changed fopen to open in this test. fopen created the file
if if does not exist, but open does not. This was unnoticed
during local testing because lit is not hermetic and reuses
files from previous runs, but it started failing on bots.
Fix the open call.

Diff Detail

Event Timeline

dvyukov requested review of this revision.Oct 27 2021, 9:03 AM
dvyukov created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptOct 27 2021, 9:03 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
melver accepted this revision.Oct 27 2021, 9:15 AM
This revision is now accepted and ready to land.Oct 27 2021, 9:15 AM
This revision was landed with ongoing or failed builds.Oct 27 2021, 9:17 AM
This revision was automatically updated to reflect the committed changes.