aboutsummaryrefslogtreecommitdiff
path: root/common/macresman.cpp
diff options
context:
space:
mode:
authorMatthew Hoops2012-12-13 00:02:22 -0500
committerMatthew Hoops2012-12-13 00:39:48 -0500
commit6b6e6c92c7c797560745651c1b16f1b5764bf297 (patch)
treedb45defb89087dba95fe70e71586c6993d0b2975 /common/macresman.cpp
parent11b920d8aedbe9ea77c83883915e4aec5b03bf76 (diff)
downloadscummvm-rg350-6b6e6c92c7c797560745651c1b16f1b5764bf297.tar.gz
scummvm-rg350-6b6e6c92c7c797560745651c1b16f1b5764bf297.tar.bz2
scummvm-rg350-6b6e6c92c7c797560745651c1b16f1b5764bf297.zip
COMMON: Fix potential uninitialized memory usage
Diffstat (limited to 'common/macresman.cpp')
-rw-r--r--common/macresman.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/common/macresman.cpp b/common/macresman.cpp
index f2f020c6de..00562f746a 100644
--- a/common/macresman.cpp
+++ b/common/macresman.cpp
@@ -360,8 +360,8 @@ bool MacResManager::load(SeekableReadStream &stream) {
_mapLength = stream.readUint32BE();
// do sanity check
- if (_dataOffset >= (uint32)stream.size() || _mapOffset >= (uint32)stream.size() ||
- _dataLength + _mapLength > (uint32)stream.size()) {
+ if (stream.eos() || _dataOffset >= (uint32)stream.size() || _mapOffset >= (uint32)stream.size() ||
+ _dataLength + _mapLength > (uint32)stream.size()) {
_resForkOffset = -1;
_mode = kResForkNone;
return false;