Adium

Ticket #13251 (closed crash: fixed)

Opened 5 months ago

Last modified 5 months ago

Page up in the scrollbarless message view crashes Adium

Reported by: piotrekm Owned by:
Milestone: Component: Message View
Version: 1.3.7 Severity: normal
Keywords: Cc:
Patch Status:

Description

When a theme which disables scrollbars (like Pretty Simple,  http://adiumxtras.com/index.php?a=xtras&xtra_id=6938) is set, pressing page up causes Adium to crash. I attach the crash report generated by the OS.

Attachments

crashlog.txt (75.7 KB) - added by piotrekm 5 months ago.
Mac OS X - generated crash log

Change History

Changed 5 months ago by piotrekm

Mac OS X - generated crash log

Changed 5 months ago by wbowling

Easily reproducible using the provided steps with either Page Up or Page Down.

Seems to get stuck in a loop where keyDown event is continually forwarded to the next responder (which doesn't change) until it crashes with a malloc error.

I found a way that seems to fix problem, but maybe it's too simple and doesn't solve everything :)

 http://bitbucket.org/wbowling/adium/changeset/4e98e1f66bcb/

Changed 5 months ago by Zachary West <zacw@…>

  • status changed from new to closed
  • resolution set to fixed

(In  6fa58273701e) Prevent infinitely forwarding navigation events. Fixes #13251.

Changed 5 months ago by Zachary West <zacw@…>

(In  ecda51fe4614) Prevent infinitely forwarding navigation events. Fixes #13251.

Note: See TracTickets for help on using tickets.