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

Adhemerval Zanella via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 8 14:26:32 PDT 2017


zatrazz added inline comments.


================
Comment at: lib/msan/tests/msan_test.cc:2213
+   ~TempFstabFile() {
+     if (fd > 0)
+       close (fd);
----------------
eugenis wrote:
> if (fd != -1)
> 
> 0 is a valid file descriptor
Indeed, I will fix it.


https://reviews.llvm.org/D32800





More information about the llvm-commits mailing list