Export of internal Abseil changes

--
5315e7b98905922e779798f3168d98343438c134 by Derek Mauro <dmauro@google.com>:

Fix absl::string_view::copy to throw std::out_of_range when pos > size().

Fixes https://github.com/abseil/abseil-cpp/issues/362

PiperOrigin-RevId: 261907364
GitOrigin-RevId: 5315e7b98905922e779798f3168d98343438c134
Change-Id: Ia8ab971c54f287411f6ea4b99f9c666c989c33fd
This commit is contained in:
Abseil Team 2019-08-06 07:13:35 -07:00 committed by CJ Johnson
parent c5c4db4f51
commit 67222ffc4c
3 changed files with 17 additions and 13 deletions

View file

@ -77,18 +77,6 @@ std::ostream& operator<<(std::ostream& o, string_view piece) {
return o; return o;
} }
string_view::size_type string_view::copy(char* buf, size_type n,
size_type pos) const {
size_type ulen = length_;
assert(pos <= ulen);
size_type rlen = std::min(ulen - pos, n);
if (rlen > 0) {
const char* start = ptr_ + pos;
std::copy(start, start + rlen, buf);
}
return rlen;
}
string_view::size_type string_view::find(string_view s, size_type pos) const string_view::size_type string_view::find(string_view s, size_type pos) const
noexcept { noexcept {
if (empty() || pos > length_) { if (empty() || pos > length_) {

View file

@ -50,6 +50,7 @@ using std::string_view;
#include "absl/base/internal/throw_delegate.h" #include "absl/base/internal/throw_delegate.h"
#include "absl/base/macros.h" #include "absl/base/macros.h"
#include "absl/base/optimization.h"
#include "absl/base/port.h" #include "absl/base/port.h"
namespace absl { namespace absl {
@ -334,7 +335,17 @@ class string_view {
// //
// Copies the contents of the `string_view` at offset `pos` and length `n` // Copies the contents of the `string_view` at offset `pos` and length `n`
// into `buf`. // into `buf`.
size_type copy(char* buf, size_type n, size_type pos = 0) const; size_type copy(char* buf, size_type n, size_type pos = 0) const {
if (ABSL_PREDICT_FALSE(pos > length_)) {
base_internal::ThrowStdOutOfRange("absl::string_view::copy");
}
size_type rlen = (std::min)(length_ - pos, n);
if (rlen > 0) {
const char* start = ptr_ + pos;
std::copy(start, start + rlen, buf);
}
return rlen;
}
// string_view::substr() // string_view::substr()
// //

View file

@ -369,6 +369,11 @@ TEST(StringViewTest, STL1) {
EXPECT_EQ(buf[1], c[1]); EXPECT_EQ(buf[1], c[1]);
EXPECT_EQ(buf[2], c[2]); EXPECT_EQ(buf[2], c[2]);
EXPECT_EQ(buf[3], a[3]); EXPECT_EQ(buf[3], a[3]);
#ifdef ABSL_HAVE_EXCEPTIONS
EXPECT_THROW(a.copy(buf, 1, 27), std::out_of_range);
#else
EXPECT_DEATH(a.copy(buf, 1, 27), "absl::string_view::copy");
#endif
} }
// Separated from STL1() because some compilers produce an overly // Separated from STL1() because some compilers produce an overly