diff options
| author | Max Horn | 2003-01-17 16:49:41 +0000 |
|---|---|---|
| committer | Max Horn | 2003-01-17 16:49:41 +0000 |
| commit | 18607b346aef99c48cdb2b3e6084a7dceb43f516 (patch) | |
| tree | bb4ac9672dc30e789f4c8ec433c5395697cd2904 /scumm/bundle.cpp | |
| parent | 375ca05ea8acefeb51b4bc4ab35f2736c23c48b0 (diff) | |
| download | scummvm-rg350-18607b346aef99c48cdb2b3e6084a7dceb43f516.tar.gz scummvm-rg350-18607b346aef99c48cdb2b3e6084a7dceb43f516.tar.bz2 scummvm-rg350-18607b346aef99c48cdb2b3e6084a7dceb43f516.zip | |
fixed OOB access introduced in my last commit; leaving in my debug asserts this time, in case there are more
svn-id: r6490
Diffstat (limited to 'scumm/bundle.cpp')
| -rw-r--r-- | scumm/bundle.cpp | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/scumm/bundle.cpp b/scumm/bundle.cpp index 8ccb906802..9d8fc0793c 100644 --- a/scumm/bundle.cpp +++ b/scumm/bundle.cpp @@ -798,10 +798,14 @@ int32 Bundle::decompressCodec(int32 codec, byte *comp_input, byte *comp_output, byte var3b; int32 adder; + byte *endPos = comp_input + input_size; + src = comp_input; memset (comp_output, 0, 0x2000); firstWord = READ_BE_UINT16(src); src += 2; + assert(src < endPos); + if (firstWord != 0) { if (index != 0) { @@ -835,6 +839,7 @@ int32 Bundle::decompressCodec(int32 codec, byte *comp_input, byte *comp_output, startPos = 0; origLeft = 0x2000; } + assert(src < endPos); tableEntrySum = 0; for (channel = 0; channel < channels; channel++) { @@ -847,13 +852,17 @@ int32 Bundle::decompressCodec(int32 codec, byte *comp_input, byte *comp_output, outputWord = 0; imcTableEntry = 7; } - left = origLeft / (2 * channels); + left = (origLeft - 1) / (2 * channels) + 1; destPos = startPos + 2 * channel; while (left--) { curTableEntry = _destImcTable[curTablePos]; decompTable = curTableEntry - 2; var3b = (1 << decompTable) << 1; readPos = src + (tableEntrySum >> 3); + if (readPos >= endPos) { + error("readPos exceeds endPos: %d >= %d (%d, %d)!" , readPos, endPos, left, origLeft); + } + assert(readPos < endPos); readWord = (uint16)(READ_BE_UINT16(readPos) << (tableEntrySum & 7)); otherTablePos = (byte)(readWord >> (16 - curTableEntry)); tableEntrySum += curTableEntry; @@ -894,13 +903,11 @@ int32 Bundle::decompressCodec(int32 codec, byte *comp_input, byte *comp_output, imcTableEntry = imcTable1[curTablePos]; } } - if (index == 0) { output_size = 0x2000 - firstWord; } else { output_size = 0x2000; } - } break; default: |