diff options
author | Paul Gilbert | 2017-11-17 20:52:18 -0500 |
---|---|---|
committer | Paul Gilbert | 2017-11-17 20:52:18 -0500 |
commit | 7361da9fdcad566a46d24e2b32711173039482d5 (patch) | |
tree | 859d8e541429c8d2024c8e6cbbb250e759d29a94 /engines/xeen/party.cpp | |
parent | ef8cf2a543ad04e971c7b09290f2d46ca0a3a4eb (diff) | |
download | scummvm-rg350-7361da9fdcad566a46d24e2b32711173039482d5.tar.gz scummvm-rg350-7361da9fdcad566a46d24e2b32711173039482d5.tar.bz2 scummvm-rg350-7361da9fdcad566a46d24e2b32711173039482d5.zip |
XEEN: Fix Coverity identified problems
Diffstat (limited to 'engines/xeen/party.cpp')
-rw-r--r-- | engines/xeen/party.cpp | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/engines/xeen/party.cpp b/engines/xeen/party.cpp index 71b1cb38c1..25a7c51e94 100644 --- a/engines/xeen/party.cpp +++ b/engines/xeen/party.cpp @@ -288,8 +288,10 @@ void Party::changeTime(int numMinutes) { !player._conditions[ERADICATED]) { for (int statNum = 0; statNum < TOTAL_STATS; ++statNum) { int statVal = player.getStat((Attribute)statNum); - if (statVal < 1) + if (statVal < 1) { player._conditions[DEAD] = 1; + killed = true; + } } } @@ -1036,6 +1038,7 @@ bool Party::giveTake(int takeMode, uint takeVal, int giveMode, uint giveVal, int ps._tempAge += giveVal; break; case 13: + assert(giveVal < 18); ps._skills[giveVal]++; break; case 15: @@ -1056,6 +1059,7 @@ bool Party::giveTake(int takeMode, uint takeVal, int giveMode, uint giveVal, int } else if (giveVal == 6) { ps._conditions[giveVal] = 1; } else { + assert(giveVal < 16); ps._conditions[giveVal]++; } @@ -1368,12 +1372,15 @@ bool Party::giveTake(int takeMode, uint takeVal, int giveMode, uint giveVal, int _gold += _vm->getRandomNumber(1, giveVal); break; case 103: + assert(takeVal < 128); _worldFlags[takeVal] = true; break; case 104: + assert(giveVal < 32); _quests[files._isDarkCc][giveVal] = true; break; case 107: + assert(takeVal < 24); _characterFlags[ps._rosterId][takeVal] = true; break; default: |