[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