CAVEPACKER: AddressSanitizer heap-buffer-overflow in BoardState #34

mgerhardy · Jan 3, 2016 · 9d8dbc7 · 9d8dbc7
1 parent 56ba5fc
commit 9d8dbc7
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/cavepacker/shared/BoardState.h b/src/cavepacker/shared/BoardState.h
@@ -107,6 +107,8 @@ class BoardState {
 	 * @return @c false if the conversion failed because the index isn't part of the board
 	 */
 	inline bool getColRowFromIndex(int index, int& col, int& row) const {
+		if (index < 0 || index >= _state.size())
+			return false;
 		if (_state[index] == '\0')
 			return false;
 		col = index % _width;