PrebufferedStream: avoid dereferencing end()

This appeared to work, but probably isn't a good idea.

CID 716940: Using invalid iterator (INVALIDATE_ITERATOR)
At (4): Dereferencing iterator "this->m_overflow.end()" though it is already past the end of its container.

Signed-off-by: Sage Weil <sage@inktank.com>

diff --git a/src/common/PrebufferedStreambuf.cc b/src/common/PrebufferedStreambuf.cc
index 600b3fd3c3691f5514dd321ff5ef525ca67d667b..d6db8bbe0af219e599739670567773792574bcfc 100644 (file)
--- a/src/common/PrebufferedStreambuf.cc
+++ b/src/common/PrebufferedStreambuf.cc
@@ -20,7 +20,7 @@ PrebufferedStreambuf::int_type PrebufferedStreambuf::overflow(int_type c)
     m_overflow.resize(old_len * 2);
   }
   m_overflow[old_len] = c;
-  this->setp(&m_overflow[old_len + 1], &*m_overflow.end());
+  this->setp(&m_overflow[old_len + 1], &*m_overflow.begin() + m_overflow.size());
   return std::char_traits<char>::not_eof(c);
 }