[Scummvm-tracker] [ScummVM :: Bugs] #12850: Unable to save games in some (all?) engines

[ScummVM :: Bugs]

#12850: Unable to save games in some (all?) engines
--------------------+------------------------
Reporter:  criezy   |       Owner:  (none)
    Type:  defect   |      Status:  new
Priority:  blocker  |   Component:  --Unset--
 Version:           |  Resolution:
Keywords:           |        Game:
--------------------+------------------------
Comment (by criezy):

 The backtrace for the assert was:
 {{{
 * thread #1, queue = 'com.apple.main-thread', stop reason = hit program
 assert
     frame #4: 0x00000001096b93c0
 scummvm`Common::Array<Common::U32String>::operator[](this=0x000000011fc50640,
 idx=4294967295) const at array.h:244:3
    241
    242          /** Return a const reference to the element at the given
 position in the array. */
    243          const T &operator[](size_type idx) const {
 -> 244                  assert(idx < _size);
    245                  return _storage[idx];
    246          }
    247
 Target 0: (scummvm) stopped.
 (lldb) bt
 * thread #1, queue = 'com.apple.main-thread', stop reason = hit program
 assert
   * frame #4: Common::Array<Common::U32String>::operator[](this,
 idx=4294967295) const at array.h:244:3
     frame #5: GUI::ListWidget::getSelectedString(this) const at
 list.h:109:55
     frame #6: GUI::SaveLoadChooserSimple::handleCommand(this, sender, cmd,
 data=15) at saveload-dialog.cpp:469:27
     frame #7: GUI::CommandSender::sendCommand(this, cmd, data=15) at
 object.h:55:13
     frame #8: GUI::ListWidget::handleMouseUp(this, x=45, y=81, button=1,
 clickCount=2) at list.cpp:269:3
     frame #9: GUI::Dialog::handleMouseUp(this, x=53, y=109, button=1,
 clickCount=2) at dialog.cpp:230:6
     frame #10: GUI::GuiManager::processEvent(this, event, activeDialog) at
 gui-manager.cpp:694:17
     frame #11: GUI::GuiManager::runLoop(this) at gui-manager.cpp:430:4
     frame #12: GUI::Dialog::runModal(this) at dialog.cpp:79:8
 }}}

 The issue is that the `ListWidget` selected item is -1 (which becomes
 4294967295 when casted to a `size_type).
-- 
Ticket URL: <https://bugs.scummvm.org/ticket/12850#comment:6>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM