Opened 13 months ago

Closed 13 months ago

Last modified 13 months ago

#14404 closed defect (fixed)

GUI: Mouse-wheel crashes save dialog

Reported by: darkshoxx Owned by: criezy
Priority: normal Component: GUI
Version: Keywords: crash savegame bug cat
Cc: darkshoxx Game: Gabriel Knight 2

Description

While trying to save in GK2, I noticed my mouse was locked within ScummVM, and (because of a Script I run in the background to limit the damage of my cat running over the keyboard) my keyboard was locked from inputs. I panicked a little and mashed mousebuttons. Caused the game to crash.
ScummVM verison 2.7.0

Was able to easily reproduce right away.

I've added the logs of both crashes, but they say very little.

Attachments (2)

scummvm.log (525 bytes ) - added by darkshoxx 13 months ago.
scummvm_old.log (525 bytes ) - added by darkshoxx 13 months ago.

Download all attachments as: .zip

Change History (7)

by darkshoxx, 13 months ago

Attachment: scummvm.log added

by darkshoxx, 13 months ago

Attachment: scummvm_old.log added

comment:1 by darkshoxx, 13 months ago

VOD:
https://www.twitch.tv/videos/1791778791?t=00h53m06s
Timestamp 53:06 is when it starts.

comment:2 by sluicebox, 13 months ago

Component: Engine: SCIGUI
Summary: Gabriel Knight 2 Save Game CrashGUI: Mouse-wheel crashes save dialog
Version: gsoc

This is a bug in the ScummVM GUI save dialog where you save a game. If you scroll the mouse wheel on this screen, it locks up crashes. I assume all engines are affected, I also tried it on an AGI game.

Great catch, thanks for the video! I was only able to figure out the cause because I heard your frantic wheel spinning; up until then you seemed locked out because of your setup, but the wheel corresponded with the final crash.

comment:3 by sluicebox, 13 months ago

The bug appeared in release 2.6.0, 2.5.1 is okay.

comment:4 by criezy, 13 months ago

Owner: set to criezy
Resolution: fixed
Status: newclosed

Thank you. This is fixed by commit 059781c.

This was a regression from commit 325260f (see also ticket #13106).

comment:5 by sluicebox, 13 months ago

Keywords: cat added
Note: See TracTickets for help on using tickets.