diff options
| author | Nick Lewycky <nicholas@mxc.ca> | 2011-08-28 03:30:02 +0000 |
|---|---|---|
| committer | Nick Lewycky <nicholas@mxc.ca> | 2011-08-28 03:30:02 +0000 |
| commit | 7bfd86d0468d276e2ed97635e910418986b8bf73 (patch) | |
| tree | d54cb69735f3d5f1108650af2e4f80bae33f5ad6 /llvm/lib/Support | |
| parent | 1fad6631265e297f8d7db10011e1b34819efd99a (diff) | |
| download | bcm5719-llvm-7bfd86d0468d276e2ed97635e910418986b8bf73.tar.gz bcm5719-llvm-7bfd86d0468d276e2ed97635e910418986b8bf73.zip | |
Fix integer overflow bug in raw_ostream::write. This showed up as a
non-deterministic crash in the test suite. Fixes PR10055!
llvm-svn: 138717
Diffstat (limited to 'llvm/lib/Support')
| -rw-r--r-- | llvm/lib/Support/raw_ostream.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Support/raw_ostream.cpp b/llvm/lib/Support/raw_ostream.cpp index 5a71fa3d8ce..c9cf249500d 100644 --- a/llvm/lib/Support/raw_ostream.cpp +++ b/llvm/lib/Support/raw_ostream.cpp @@ -84,7 +84,7 @@ void raw_ostream::SetBuffered() { } void raw_ostream::SetBufferAndMode(char *BufferStart, size_t Size, - BufferKind Mode) { + BufferKind Mode) { assert(((Mode == Unbuffered && BufferStart == 0 && Size == 0) || (Mode != Unbuffered && BufferStart && Size)) && "stream must be unbuffered or have at least one byte"); @@ -284,7 +284,7 @@ raw_ostream &raw_ostream::write(unsigned char C) { raw_ostream &raw_ostream::write(const char *Ptr, size_t Size) { // Group exceptional cases into a single branch. - if (BUILTIN_EXPECT(OutBufCur+Size > OutBufEnd, false)) { + if (BUILTIN_EXPECT(size_t(OutBufEnd - OutBufCur) < Size, false)) { if (BUILTIN_EXPECT(!OutBufStart, false)) { if (BufferMode == Unbuffered) { write_impl(Ptr, Size); |

