Opened 11 months ago
Closed 11 months ago
#18716 closed bug (fixed)
Crash in BListView::MouseUp()
Reported by: | jackburton | Owned by: | nobody |
---|---|---|---|
Priority: | normal | Milestone: | R1/beta5 |
Component: | Kits/Interface Kit | Version: | R1/Development |
Keywords: | BListView | Cc: | |
Blocked By: | Blocking: | ||
Platform: | All |
Description
The attached application crashes in BListView::MouseUp().
Seems the problem could be the changes in https://cgit.haiku-os.org/haiku/commit/?id=6761bf581fd14cac9fd22825fa6baa399263dc83
Specifically, these lines:
- uint32* buttons = 0; - GetMouse(&where, buttons); + // if mouse up selection is invalid reselect mouse down selection + if (index == -1) + index = fTrack->item_index;
where index can be set to an invalid item (0) in certain cases. I encountered the bug myself in Genio. The problem was debugged and tracked down by Andrea Anzani, which also provided the test application.
For reference, here's a link to the Genio issue: https://github.com/Genio-The-Haiku-IDE/Genio/issues/228
Attachments (1)
Change History (3)
by , 11 months ago
Attachment: | BListViewCrash.zip added |
---|
comment:1 by , 11 months ago
Milestone: | Unscheduled → R1/beta5 |
---|
comment:2 by , 11 months ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Fix merged in hrev57476.