r207137 - Fix a tiny test-only leak, found by LSan.

Nico Weber nicolasweber at gmx.de
Thu Apr 24 12:04:11 PDT 2014


Author: nico
Date: Thu Apr 24 14:04:10 2014
New Revision: 207137

URL: http://llvm.org/viewvc/llvm-project?rev=207137&view=rev
Log:
Fix a tiny test-only leak, found by LSan.

Modified:
    cfe/trunk/unittests/libclang/LibclangTest.cpp

Modified: cfe/trunk/unittests/libclang/LibclangTest.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/unittests/libclang/LibclangTest.cpp?rev=207137&r1=207136&r2=207137&view=diff
==============================================================================
--- cfe/trunk/unittests/libclang/LibclangTest.cpp (original)
+++ cfe/trunk/unittests/libclang/LibclangTest.cpp Thu Apr 24 14:04:10 2014
@@ -49,14 +49,14 @@ struct TestVFO {
   }
 
   ~TestVFO() {
-    if (!Contents)
-      return;
-    char *BufPtr;
-    unsigned BufSize;
-    clang_VirtualFileOverlay_writeToBuffer(VFO, 0, &BufPtr, &BufSize);
-    std::string BufStr(BufPtr, BufSize);
-    EXPECT_STREQ(Contents, BufStr.c_str());
-    free(BufPtr);
+    if (Contents) {
+      char *BufPtr;
+      unsigned BufSize;
+      clang_VirtualFileOverlay_writeToBuffer(VFO, 0, &BufPtr, &BufSize);
+      std::string BufStr(BufPtr, BufSize);
+      EXPECT_STREQ(Contents, BufStr.c_str());
+      free(BufPtr);
+    }
     clang_VirtualFileOverlay_dispose(VFO);
   }
 };





More information about the cfe-commits mailing list