[Scummvm-tracker] [ScummVM :: Bugs] #13832: GROOVIE: Bug #2 possible crash with incorrect seeks to endpos in roq.cpp

[ScummVM :: Bugs]

#13832: GROOVIE: Bug #2 possible crash with incorrect seeks to endpos in roq.cpp
---------------------+------------------------------
Reporter:  lolovo    |       Owner:  (none)
    Type:  defect    |      Status:  new
Priority:  normal    |   Component:  Engine: Groovie
 Version:            |  Resolution:
Keywords:  groovie2  |        Game:  11th Hour
---------------------+------------------------------
Description changed by lolovo:

Old description:

> Same bug, related to this issue - https://bugs.scummvm.org/ticket/13809
>
> Now inside of ROQPlayer::processBlockQuadVector, when after travers the
> image we've got wrong skipBytes value that lead to crash on assertion in
> seek. In this case we need return false and break (without break
> artifacts will occur) and appropriate warning will be in logs.
>
> Crash during start egypt pyramid word puzzle.
>
> Fix will be provided in PR.

New description:

 Same bug, related to this issue - https://bugs.scummvm.org/ticket/13809

 Now inside of ROQPlayer::processBlockQuadVector, when after travers the
 image we've got wrong skipBytes value that lead to crash on assertion in
 seek. In this case we need return false and break (without break artifacts
 will occur) and appropriate warning will be in logs.

 Crash during start egypt pyramid word puzzle.

 Fix will be provided in PR.

 P.S. Save file is provided for proof (right before puzzle on front).

--
-- 
Ticket URL: <https://bugs.scummvm.org/ticket/13832#comment:1>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM