[PATCH] D32800: [msan] Fix getmntent{_r} for empty /etc/fstab

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 8 14:03:31 PDT 2017


eugenis accepted this revision.
eugenis added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/msan/tests/msan_test.cc:2213
+   ~TempFstabFile() {
+     if (fd > 0)
+       close (fd);
----------------
if (fd != -1)

0 is a valid file descriptor


https://reviews.llvm.org/D32800





More information about the llvm-commits mailing list