nerdcruft/libcruft-util
1
0
Fork 0
Code Issues Releases Activity

view: check the index is valid for the subscript operator

Browse Source
This commit is contained in:
Danny Robson 2019-11-05 14:16:31 +11:00
parent b479905b12
commit 762678a50f
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
view.hpp
View File

@ -340,7 +340,7 @@ namespace cruft {
[[nodiscard]] constexpr auto [[nodiscard]] constexpr auto
split (IndexT idx) const split (IndexT idx) const
{ {
CHECK_LE (cruft::cast::lossless<std::size_t> (idx), size ()); CHECK_LIMIT (idx, IndexT {0u}, cruft::cast::lossless<IndexT> (size ()));
auto last = m_begin; auto last = m_begin;
std::advance (last, idx); std::advance (last, idx);
@ -479,6 +479,7 @@ namespace cruft {
constexpr auto&& constexpr auto&&
operator[] (size_t idx) noexcept operator[] (size_t idx) noexcept
{ {
CHECK_LIMIT (idx, 0u, size ());
return *std::next (begin (), idx); return *std::next (begin (), idx);
} }
@ -486,6 +487,7 @@ namespace cruft {
constexpr auto&& constexpr auto&&
operator[] (size_t idx) const noexcept operator[] (size_t idx) const noexcept
{ {
CHECK_LIMIT (idx, 0u, size ());
return *std::next (begin (), idx); return *std::next (begin (), idx);
} }