aboutsummaryrefslogtreecommitdiff
path: root/engines/xeen/party.cpp
diff options
context:
space:
mode:
authorPaul Gilbert2017-11-17 20:52:18 -0500
committerPaul Gilbert2017-11-17 20:52:18 -0500
commit7361da9fdcad566a46d24e2b32711173039482d5 (patch)
tree859d8e541429c8d2024c8e6cbbb250e759d29a94 /engines/xeen/party.cpp
parentef8cf2a543ad04e971c7b09290f2d46ca0a3a4eb (diff)
downloadscummvm-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.cpp9
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: