From 181f0bdf720f85f29c3c159b7f77d70b00dc22cc Mon Sep 17 00:00:00 2001
From: silverflyone <silver.fly.one@gmail.com>
Date: Tue, 4 Apr 2023 03:27:32 +1000
Subject: [PATCH] Update Buildings.cpp

Boundary checks added.
---
 library/modules/Buildings.cpp | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/library/modules/Buildings.cpp b/library/modules/Buildings.cpp
index 8a71485b2..172975e21 100644
--- a/library/modules/Buildings.cpp
+++ b/library/modules/Buildings.cpp
@@ -1635,6 +1635,12 @@ StockpileIterator& StockpileIterator::operator++() {
         if (block) {
             // Check the next item in the current block.
             ++current;
+        }
+        else if (stockpile->x2 < 0 || stockpile->y2 < 0 || stockpile->z < 0 || stockpile->x1 > world->map.x_count - 1 || stockpile->y1 > world->map.y_count - 1 || stockpile->z > world->map.z_count - 1) {
+            // if the stockpile bounds exist outside of valid map plane then no items can be in the stockpile
+            block = NULL;
+            item = NULL;
+            return *this;
         } else {
             // Start with the top-left block covering the stockpile.
             block = Maps::getTileBlock(std::min(std::max(stockpile->x1, 0), world->map.x_count-1), std::min(std::max(stockpile->y1, 0), world->map.y_count-1), stockpile->z);