Opened 2 years ago

Closed 2 years ago

#13339 closed defect (fixed)

GUI: Game Options crashes launcher at small application widths

Reported by: antoniou79 Owned by: antoniou79
Priority: normal Component: GUI
Version: Keywords: crash, assertion fault, width, resize
Cc: Game:


Tested on Windows 10 x64 (Version 10.0.19044 Build 19044, all updates) built from latest master (HEAD).

Steps to reproduce:

  • Launch ScummVM in windowed mode
  • From the launcher select any game from the list
  • Click on "Game Options..." (former "Edit Game...") button.
  • When the dialogue opens, try to resize the ScummVM's window borders so as to reduce the window's width to a very small size. (see attached screenshot for example)

ScummVM crashes with the message:

Assertion failed: isValidRect(), file ./common/rect.h, line 160

Expected Result:
ScummVM should handle this without crashing.

The crash does not occur for the launcher's "Global Options...", so it seems to be specific to the "Game Options..." dialogue.

If the width of the launcher is already small when launching ScummVM, then the crash will occur immediately upon clicking on "Game Options..."

A quick debugging session with VS indicates that the fault lies somewhere in ScrollContainerWidget::reflowLayout() in gui\widgets\scrollcontainer.cpp during the loop of "reflow layout of inner widgets"

From the related discussion on Discord, it seems that the issue has been around for a while (quite possibly since 2.2.0, according to rootfather)

Attachments (1)

smallWidth.png (182.0 KB ) - added by antoniou79 2 years ago.

Download all attachments as: .zip

Change History (4)

by antoniou79, 2 years ago

Attachment: smallWidth.png added

comment:1 by antoniou79, 2 years ago

Further looking into this, it seems that the culprit (or one of them) is EditTextWidget::getEditRect() from edittext.cpp and in particular this line:

Common::Rect r(2 + _leftPadding, 1, _w - 1 - _rightPadding, _h);

which, for low widths, attempts to create a Rect for which the right x coordinate is lower than the left x coordinate -- and this triggers the assertion fault in the isValidRect() check.

EditTextWidget::getEditRect() is called by EditableWidget::reflowLayout() at this line:

	_editScrollOffset = g_gui.getStringWidth(_editString, _font) - getEditRect().width();

The Game Options "Game" tab has two editable fields, so they are the ones causing the crash.
In very low widths, maybe these fields should not show up at all (just to prevent the crash)?

Last edited 2 years ago by antoniou79 (previous) (diff)

comment:2 by antoniou79, 2 years ago

I've implemented a small fix for this, based on similar code in our listWidget (list.cpp); see:

Pull Request here:

comment:3 by antoniou79, 2 years ago

Owner: set to antoniou79
Resolution: fixed
Status: newclosed

The PR was merged, so this ticket can be closed.

Note: See TracTickets for help on using tickets.