[Scummvm-tracker] [ScummVM] #10219: MOHAWK: Riven: Invalid read for RivenVideo::play/load

[dafioram]

#10219: MOHAWK: Riven: Invalid read for RivenVideo::play/load
-----------------------+----------------------------
  Reporter:  dafioram  |      Owner:  (none)
      Type:  defect    |     Status:  new
  Priority:  normal    |  Component:  Engine: Mohawk
Resolution:            |   Keywords:
      Game:  Riven     |
-----------------------+----------------------------

Comment (by dafioram):

 A similar looking stack trace also happens when pressing the buttons on
 the star fissure hatch door.

 {{{
 ==25483== Invalid read of size 4
 ==25483==    at 0x36E90D:
 Audio::QuickTimeAudioDecoder::QuickTimeAudioTrack::seek(Audio::Timestamp
 const&) (quicktime.cpp:327)
 ==25483==    by 0x29F386:
 Video::VideoDecoder::SeekableAudioTrack::seek(Audio::Timestamp const&)
 (video_decoder.cpp:693)
 ==25483==    by 0x29E69B: Video::VideoDecoder::Track::rewind()
 (video_decoder.cpp:530)
 ==25483==    by 0x29DABC: Video::VideoDecoder::rewind()
 (video_decoder.cpp:323)
 ==25483==    by 0x1CC462: Mohawk::RivenVideo::play() (riven_video.cpp:276)
 ==25483==    by 0x1C1F73: Mohawk::RivenSimpleCommand::playMovie(unsigned
 short, Common::Array<unsigned short> const&) (riven_scripts.cpp:648)
 ==25483==    by 0x1C2EC9: Mohawk::RivenSimpleCommand::execute()
 (riven_scripts.cpp:794)
 ==25483==    by 0x1C0A22:
 Mohawk::RivenScript::run(Mohawk::RivenScriptManager*)
 (riven_scripts.cpp:226)
 ==25483==    by 0x1C0421: Mohawk::RivenScriptManager::runQueuedScripts()
 (riven_scripts.cpp:141)
 ==25483==    by 0x1AACA7: Mohawk::MohawkEngine_Riven::doFrame()
 (riven.cpp:272)
 ==25483==    by 0x1AA7EF: Mohawk::MohawkEngine_Riven::run()
 (riven.cpp:196)
 ==25483==    by 0x1643D8: runGame(PluginSubclass<MetaEngine> const*,
 OSystem&, Common::String const&) (main.cpp:263)
 ==25483==  Address 0x1a9fb66c is 8 bytes after a block of size 20 alloc'd
 ==25483==    at 0x4C2E8BF: operator new[](unsigned long) (in
 /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
 ==25483==    by 0x3BE47E:
 Common::QuickTimeParser::readELST(Common::QuickTimeParser::Atom)
 (quicktime.cpp:438)
 ==25483==    by 0x3BD424:
 Common::QuickTimeParser::readDefault(Common::QuickTimeParser::Atom)
 (quicktime.cpp:232)
 ==25483==    by 0x3BD424:
 Common::QuickTimeParser::readDefault(Common::QuickTimeParser::Atom)
 (quicktime.cpp:232)
 ==25483==    by 0x3BDE6D:
 Common::QuickTimeParser::readTRAK(Common::QuickTimeParser::Atom)
 (quicktime.cpp:375)
 ==25483==    by 0x3BD424:
 Common::QuickTimeParser::readDefault(Common::QuickTimeParser::Atom)
 (quicktime.cpp:232)
 ==25483==    by 0x3BD575:
 Common::QuickTimeParser::readMOOV(Common::QuickTimeParser::Atom)
 (quicktime.cpp:258)
 ==25483==    by 0x3BD424:
 Common::QuickTimeParser::readDefault(Common::QuickTimeParser::Atom)
 (quicktime.cpp:232)
 ==25483==    by 0x3BCC72:
 Common::QuickTimeParser::parseStream(Common::SeekableReadStream*,
 DisposeAfterUse::Flag) (quicktime.cpp:104)
 ==25483==    by 0x298D00:
 Video::QuickTimeDecoder::loadStream(Common::SeekableReadStream*)
 (qt_decoder.cpp:68)
 ==25483==    by 0x1CB73C: Mohawk::RivenVideo::load(unsigned short)
 (riven_video.cpp:67)
 ==25483==    by 0x1C26BA:
 Mohawk::RivenSimpleCommand::activateMLST(Mohawk::MLSTRecord const&) const
 (riven_scripts.cpp:743)
 }}}

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