Fix UB in SpiralIterator::operator++

* Calling pop_back on an iterator past the end is UB Signed-off-by: Ryan Friedman <[email protected]>
ANYbotics · Nov 8, 2024 · 78b5ceb · 78b5ceb
1 parent 38e57e2
commit 78b5ceb
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/grid_map_core/src/iterators/SpiralIterator.cpp b/grid_map_core/src/iterators/SpiralIterator.cpp
@@ -66,6 +66,9 @@ const Eigen::Array2i & SpiralIterator::operator*() const
 
 SpiralIterator & SpiralIterator::operator++()
 {
+  if (isPastEnd()) {
+    return *this;
+  }
   pointsRing_.pop_back();
   if (pointsRing_.empty() && !isPastEnd()) {generateRing();}
   return *this;