diff options
| author | Nick Lewycky <nicholas@mxc.ca> | 2007-07-01 03:06:30 +0000 |
|---|---|---|
| committer | Nick Lewycky <nicholas@mxc.ca> | 2007-07-01 03:06:30 +0000 |
| commit | 5105adfcd1dea46cfbcbab9d0d8f8191cb0e2188 (patch) | |
| tree | a524c1c3d5eb4652617db6fc0b7f0bb4002fc52b /llvm/lib/Support/MemoryBuffer.cpp | |
| parent | 097bb1b72ef39b67cbcdb36d9e08dfef044e0658 (diff) | |
| download | bcm5719-llvm-5105adfcd1dea46cfbcbab9d0d8f8191cb0e2188.tar.gz bcm5719-llvm-5105adfcd1dea46cfbcbab9d0d8f8191cb0e2188.zip | |
Fix undefined behaviour reported by the new --enable-expensive-checks option.
llvm-svn: 37829
Diffstat (limited to 'llvm/lib/Support/MemoryBuffer.cpp')
| -rw-r--r-- | llvm/lib/Support/MemoryBuffer.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/lib/Support/MemoryBuffer.cpp b/llvm/lib/Support/MemoryBuffer.cpp index 0e857021a48..43eb1813ad6 100644 --- a/llvm/lib/Support/MemoryBuffer.cpp +++ b/llvm/lib/Support/MemoryBuffer.cpp @@ -251,8 +251,9 @@ MemoryBuffer *MemoryBuffer::getSTDIN() { while (size_t ReadBytes = fread(Buffer, 1, 4096*4, stdin)) FileData.insert(FileData.end(), Buffer, Buffer+ReadBytes); + FileData.push_back(0); // &FileData[Size] is invalid. So is &*FileData.end(). size_t Size = FileData.size(); MemoryBuffer *B = new STDINBufferFile(); - B->initCopyOf(&FileData[0], &FileData[Size]); + B->initCopyOf(&FileData[0], &FileData[Size-1]); return B; } |
