From ef7becc9c8a48804d3fd9dac032f7b33e561a612 Mon Sep 17 00:00:00 2001 From: Jonathan Wakely Date: Tue, 31 Aug 2021 17:34:51 +0100 Subject: [PATCH] libstdc++: Add valid range checks to std::span constructors [PR98421] Signed-off-by: Jonathan Wakely libstdc++-v3/ChangeLog: PR libstdc++/98421 * include/std/span (span(Iter, size_type), span(Iter, Iter)): Add valid range checks. * testsuite/23_containers/span/cons_1_assert_neg.cc: New test. * testsuite/23_containers/span/cons_2_assert_neg.cc: New test. --- libstdc++-v3/include/std/span | 2 ++ .../23_containers/span/cons_1_assert_neg.cc | 14 ++++++++++++++ .../23_containers/span/cons_2_assert_neg.cc | 14 ++++++++++++++ 3 files changed, 30 insertions(+) create mode 100644 libstdc++-v3/testsuite/23_containers/span/cons_1_assert_neg.cc create mode 100644 libstdc++-v3/testsuite/23_containers/span/cons_2_assert_neg.cc diff --git a/libstdc++-v3/include/std/span b/libstdc++-v3/include/std/span index 21d8f6a43a6..be053e8ef38 100644 --- a/libstdc++-v3/include/std/span +++ b/libstdc++-v3/include/std/span @@ -160,6 +160,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION { __glibcxx_assert(__count == _Extent); } + __glibcxx_requires_valid_range(__first, __first + __count); } template _End> @@ -175,6 +176,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION { __glibcxx_assert((__last - __first) == _Extent); } + __glibcxx_requires_valid_range(__first, __last); } template diff --git a/libstdc++-v3/testsuite/23_containers/span/cons_1_assert_neg.cc b/libstdc++-v3/testsuite/23_containers/span/cons_1_assert_neg.cc new file mode 100644 index 00000000000..2f555125453 --- /dev/null +++ b/libstdc++-v3/testsuite/23_containers/span/cons_1_assert_neg.cc @@ -0,0 +1,14 @@ +// { dg-options "-std=gnu++2a" } +// { dg-do run { xfail *-*-* } } +// { dg-require-effective-target c++2a } + +#undef _GLIBCXX_DEBUG +#define _GLIBCXX_DEBUG +#include +#include + +int main() +{ + std::vector v(2); + std::span s(v.begin(), 3); +} diff --git a/libstdc++-v3/testsuite/23_containers/span/cons_2_assert_neg.cc b/libstdc++-v3/testsuite/23_containers/span/cons_2_assert_neg.cc new file mode 100644 index 00000000000..efef0e608ba --- /dev/null +++ b/libstdc++-v3/testsuite/23_containers/span/cons_2_assert_neg.cc @@ -0,0 +1,14 @@ +// { dg-options "-std=gnu++2a" } +// { dg-do run { xfail *-*-* } } +// { dg-require-effective-target c++2a } + +#undef _GLIBCXX_DEBUG +#define _GLIBCXX_DEBUG +#include +#include + +int main() +{ + std::vector v(2), w(1); + std::span s(v.begin(), w.end()); +}