libretro/scummvm
1
0
Fork 0
mirror of https://github.com/libretro/scummvm.git synced 2024-12-15 22:28:10 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

SCUMM: (SCUMM 3) - avoid possible division by 0

Browse Source 
(I haven't encountered a case where it actually happened, but it looks like it could...)
This commit is contained in:
athrxx 2021-09-04 18:23:43 +02:00
parent cabb818543
commit a70206f56f
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
engines/scumm/actor.cpp
View File

@ -553,8 +553,8 @@ int Actor_v3::calcMovementFactor(const Common::Point& next) {
if (diffY < 0)
deltaYFactor = -deltaYFactor;
_walkdata.xfrac = _walkdata.xAdd = diffX / deltaXFactor;
_walkdata.yfrac = _walkdata.yAdd = diffY / deltaYFactor;
_walkdata.xfrac = _walkdata.xAdd = deltaXFactor ? diffX / deltaXFactor : 0;
_walkdata.yfrac = _walkdata.yAdd = deltaYFactor ? diffY / deltaYFactor : 0;
_walkdata.cur = _pos;
_walkdata.next = next;
_walkdata.deltaXFactor = deltaXFactor;
@ -3818,8 +3818,8 @@ void Actor_v3::saveLoadWithSerializer(Common::Serializer &s) {
_walkdata.deltaXFactor = -_walkdata.deltaXFactor;
if (diffY < 0)
_walkdata.deltaYFactor = -_walkdata.deltaYFactor;
_walkdata.xfrac = _walkdata.xAdd = diffX / _walkdata.deltaXFactor;
_walkdata.yfrac = _walkdata.yAdd = diffY / _walkdata.deltaYFactor;
_walkdata.xfrac = _walkdata.xAdd = _walkdata.deltaXFactor ? diffX / _walkdata.deltaXFactor : 0;
_walkdata.yfrac = _walkdata.yAdd = _walkdata.deltaYFactor ? diffY / _walkdata.deltaYFactor : 0;
} else {
s.syncAsUint16LE(_walkdata.xAdd, VER(rev));