From 5ac28a0cfda2c5f620886ace6f3b882e2e9a571d Mon Sep 17 00:00:00 2001 From: Marshall Clow Date: Tue, 23 Oct 2018 20:07:45 +0000 Subject: Off-by-one errors strike again. Thank goodness for ASAN and the bots. llvm-svn: 345076 --- libcxx/include/vector | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) (limited to 'libcxx/include/vector') diff --git a/libcxx/include/vector b/libcxx/include/vector index 8ac7576ae7f..196ea70eafd 100644 --- a/libcxx/include/vector +++ b/libcxx/include/vector @@ -2606,8 +2606,13 @@ vector::__construct_at_end(size_type __n, bool __x) { size_type __old_size = this->__size_; this->__size_ += __n; - if (__old_size == 0 || (__old_size / __bits_per_word) != (this->__size_ / __bits_per_word)) - this->__begin_[this->__size_ / __bits_per_word] = __storage_type(0); + if (__old_size == 0 || ((__old_size - 1) / __bits_per_word) != ((this->__size_ - 1) / __bits_per_word)) + { + if (this->__size_ <= __bits_per_word) + this->__begin_[0] = __storage_type(0); + else + this->__begin_[(this->__size_ - 1) / __bits_per_word] = __storage_type(0); + } _VSTD::fill_n(__make_iter(__old_size), __n, __x); } @@ -2622,8 +2627,13 @@ vector::__construct_at_end(_ForwardIterator __first, _ForwardI { size_type __old_size = this->__size_; this->__size_ += _VSTD::distance(__first, __last); - if (__old_size == 0 || (__old_size / __bits_per_word) != (this->__size_ / __bits_per_word)) - this->__begin_[this->__size_ / __bits_per_word] = __storage_type(0); + if (__old_size == 0 || ((__old_size - 1) / __bits_per_word) != ((this->__size_ - 1) / __bits_per_word)) + { + if (this->__size_ <= __bits_per_word) + this->__begin_[0] = __storage_type(0); + else + this->__begin_[(this->__size_ - 1) / __bits_per_word] = __storage_type(0); + } _VSTD::copy(__first, __last, __make_iter(__old_size)); } -- cgit v1.2.3