Opened 4 years ago

Closed 4 years ago

#16267 closed bug (fixed)

Input preferences crash when adjusting touchpad settings

Reported by: tatoomi Owned by: PreetpalKaur
Priority: normal Milestone: R1/beta3
Component: Preferences/Input Version: R1/beta2
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

since Hrev 54326 the touchpad utility is back -Thank you- But as soon as you want to adjust the utility 'input "crash

Attachments (1)

Input-1241-debug-15-06-2020-14-16-31.report (11.5 KB ) - added by tatoomi 4 years ago.
Report

Download all attachments as: .zip

Change History (9)

by tatoomi, 4 years ago

Report

comment:1 by waddlesplash, 4 years ago

Component: Drivers/InputPreferences/Input

comment:2 by pulkomandy, 4 years ago

Summary: input utility crashInput preferences crash when adjusting touchpad settings

comment:3 by eml, 4 years ago

I'm looking into this ticket and would like to be assigned to prevent duplicate work. I can see why it crashes: InputTouchpadPref.h the class has a BInputDevice pointer called fTouchPad, this in turn is called in InputTouchpadPref.cpp but from what I can tell this will always be a nullpointer, as no value is ever set.

I'm trying to figure out how to set it, pointers are welcome :-)

comment:5 by eml, 4 years ago

What a miss... I keep misreading the code for this application. Thank you! I see this is assigned to PreetpalKaur, but I hope it's okay by him that I'm looking at it. If nothing else it lets me dig into more code and feel some sort of success.

comment:6 by pulkomandy, 4 years ago

Yes, she is the default owner of tickets on Input preferences as the main author of it. The ticket is not marked in-progress so it's fine to work on it.

comment:7 by eml, 4 years ago

Funnily, you already have a fix for it here: https://review.haiku-os.org/c/haiku/+/1590 :)

Gerrit made me aware of that so I abandoned my patch (also you did it better anyhow.) I could extract your changes for the crash into a patch specifically for this ticket, perhaps? That way the bug gets fixed without introducing new features.

comment:8 by pulkomandy, 4 years ago

Milestone: UnscheduledR1/beta3
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.