From 7e02ec87b9d7e26e6f811bae4586636c9b880fcb Mon Sep 17 00:00:00 2001 From: "madmaxoft@gmail.com" Date: Thu, 9 May 2013 19:06:16 +0000 Subject: Fixed a few out-of-bounds reads git-svn-id: http://mc-server.googlecode.com/svn/trunk@1465 0a769ca7-a7f5-676a-18bf-c427514a06d6 --- source/Blocks/BlockDirt.h | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) (limited to 'source/Blocks') diff --git a/source/Blocks/BlockDirt.h b/source/Blocks/BlockDirt.h index 73bc28dc5..6cec5d99e 100644 --- a/source/Blocks/BlockDirt.h +++ b/source/Blocks/BlockDirt.h @@ -34,11 +34,14 @@ public: } // Grass becomes dirt if there is something on top of it: - BLOCKTYPE Above = a_World->GetBlock(a_BlockX, a_BlockY + 1, a_BlockZ); - if (!g_BlockTransparent[Above] && !g_BlockOneHitDig[Above]) + if (a_BlockY < cChunkDef::Height - 1) { - a_World->FastSetBlock(a_BlockX, a_BlockY, a_BlockZ, E_BLOCK_DIRT, 0); - return; + BLOCKTYPE Above = a_World->GetBlock(a_BlockX, a_BlockY + 1, a_BlockZ); + if (!g_BlockTransparent[Above] && !g_BlockOneHitDig[Above]) + { + a_World->FastSetBlock(a_BlockX, a_BlockY, a_BlockZ, E_BLOCK_DIRT, 0); + return; + } } // Grass spreads to adjacent blocks: @@ -51,6 +54,11 @@ public: BLOCKTYPE DestBlock; NIBBLETYPE DestMeta; + if ((a_BlockY + OfsY < 0) || (a_BlockY + OfsY >= cChunkDef::Height - 1)) + { + // Y Coord out of range + continue; + } bool IsValid = a_World->GetBlockTypeMeta(a_BlockX + OfsX, a_BlockY + OfsY, a_BlockZ + OfsZ, DestBlock, DestMeta); if (!IsValid || (DestBlock != E_BLOCK_DIRT)) { -- cgit v1.2.3