aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFrançois Dumont <fdumont@gcc.gnu.org>2024-02-20 19:24:47 +0100
committerFrançois Dumont <fdumont@gcc.gnu.org>2024-02-20 22:40:11 +0100
commitca5f509903315a56b2f306f8644544861a6525cb (patch)
treed8bec7c87b452f77a6527f2d7168a6076f54beef
parent14ba8d5b87acd5f91ab8b8c02165a0fd53dcc2f2 (diff)
downloadgcc-ca5f509903315a56b2f306f8644544861a6525cb.zip
gcc-ca5f509903315a56b2f306f8644544861a6525cb.tar.gz
gcc-ca5f509903315a56b2f306f8644544861a6525cb.tar.bz2
libstdc++: [_GLIBCXX_DEBUG] Fix std::__niter_wrap behavior
In _GLIBCXX_DEBUG mode the std::__niter_base can remove 2 layers, the __gnu_debug::_Safe_iterator<> and the __gnu_cxx::__normal_iterator<>. When std::__niter_wrap is called to build a __gnu_debug::_Safe_iterator<> from a __gnu_cxx::__normal_iterator<> we then have a consistency issue as the difference between the 2 iterators will done on a __normal_iterator on one side and a C pointer on the other. To avoid this problem call std::__niter_base on both input iterators. libstdc++-v3/ChangeLog: * include/bits/stl_algobase.h (std::__niter_wrap): Add a call to std::__niter_base on res iterator.
-rw-r--r--libstdc++-v3/include/bits/stl_algobase.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/libstdc++-v3/include/bits/stl_algobase.h b/libstdc++-v3/include/bits/stl_algobase.h
index 0f73da1..d534e02 100644
--- a/libstdc++-v3/include/bits/stl_algobase.h
+++ b/libstdc++-v3/include/bits/stl_algobase.h
@@ -344,7 +344,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
_GLIBCXX20_CONSTEXPR
inline _From
__niter_wrap(_From __from, _To __res)
- { return __from + (__res - std::__niter_base(__from)); }
+ { return __from + (std::__niter_base(__res) - std::__niter_base(__from)); }
// No need to wrap, iterator already has the right type.
template<typename _Iterator>