From: Alp Toker <alp@nuanti.com>
Date: Fri, 9 May 2014 08:57:32 +0000 (+0000)
Subject: MemoryBuffer: don't force mmap when stat fails
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=8d24d43c95eda711f1491dfd2eda91911ab91f04;p=oota-llvm.git

MemoryBuffer: don't force mmap when stat fails

Fix error handling introduced in r127426 that could result in MemoryBuffers not
having null termination.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@208396 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Support/MemoryBuffer.cpp b/lib/Support/MemoryBuffer.cpp
index f95383dfb29..26c3a2ebd26 100644
--- a/lib/Support/MemoryBuffer.cpp
+++ b/lib/Support/MemoryBuffer.cpp
@@ -320,9 +320,8 @@ static bool shouldUseMmap(int FD,
   // RequiresNullTerminator = false and MapSize != -1.
   if (FileSize == size_t(-1)) {
     sys::fs::file_status Status;
-    error_code EC = sys::fs::status(FD, Status);
-    if (EC)
-      return EC;
+    if (sys::fs::status(FD, Status))
+      return false;
     FileSize = Status.getSize();
   }