summaryrefslogtreecommitdiffstats
path: root/llvm/lib/Support
diff options
context:
space:
mode:
authorNick Lewycky <nicholas@mxc.ca>2011-08-28 03:30:02 +0000
committerNick Lewycky <nicholas@mxc.ca>2011-08-28 03:30:02 +0000
commit7bfd86d0468d276e2ed97635e910418986b8bf73 (patch)
treed54cb69735f3d5f1108650af2e4f80bae33f5ad6 /llvm/lib/Support
parent1fad6631265e297f8d7db10011e1b34819efd99a (diff)
downloadbcm5719-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.cpp4
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);
OpenPOWER on IntegriCloud