Page MenuHomePhabricator

[sanitizer_common] Add test for popen()
ClosedPublic

Authored by mgorny on Sat, Dec 29, 10:53 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

mgorny created this revision.Sat, Dec 29, 10:53 AM

It looks fine, but we don't have interceptors for popen(3), popenve(3), pclose(3). Could you include them together with this patch? Add add a test for popenve(3).

krytarowski accepted this revision.Sat, Dec 29, 11:57 AM

But of course new code can be added as a new revision.

This revision is now accepted and ready to land.Sat, Dec 29, 11:57 AM
mgorny updated this revision to Diff 179712.Sat, Dec 29, 2:49 PM

Added assertion verifying that fileno(fp) works.

This revision was automatically updated to reflect the committed changes.