diff options
author | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2011-12-10 01:38:26 +0000 |
---|---|---|
committer | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2011-12-10 01:38:26 +0000 |
commit | cc6107df087e3294bde69584aaeaae77adeac2d3 (patch) | |
tree | 406b161f15ac687aaa7019b094aa201adff6311d /clang/lib/Basic/SourceManager.cpp | |
parent | 8ca0c6408e537e70dddbd8f4d7fd11c5eb019c71 (diff) | |
download | llvm-cc6107df087e3294bde69584aaeaae77adeac2d3.zip llvm-cc6107df087e3294bde69584aaeaae77adeac2d3.tar.gz llvm-cc6107df087e3294bde69584aaeaae77adeac2d3.tar.bz2 |
In ContentCache::replaceBuffer, add sanity check to make sure that we do not free a buffer
and then continue using it. rdar://10359140.
llvm-svn: 146308
Diffstat (limited to 'clang/lib/Basic/SourceManager.cpp')
-rw-r--r-- | clang/lib/Basic/SourceManager.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/clang/lib/Basic/SourceManager.cpp b/clang/lib/Basic/SourceManager.cpp index 85fe474..18026db 100644 --- a/clang/lib/Basic/SourceManager.cpp +++ b/clang/lib/Basic/SourceManager.cpp @@ -71,7 +71,11 @@ unsigned ContentCache::getSize() const { void ContentCache::replaceBuffer(const llvm::MemoryBuffer *B, bool DoNotFree) { - assert(B != Buffer.getPointer()); + if (B == Buffer.getPointer()) { + assert(0 && "Replacing with the same buffer"); + Buffer.setInt(DoNotFree? DoNotFreeFlag : 0); + return; + } if (shouldFreeBuffer()) delete Buffer.getPointer(); |