Input: synaptics - clear device info before filling in
Subsystems (2)
PATCH 1/1 Input: synaptics - clear device info before filling in
From: Eric Biggers <[email protected]>
synaptics_query_hardware() was being passed a
'struct synaptics_device_info' in uninitialized stack memory, then not always initializing all fields. This caused garbage to show up in certain fields, making the touchpad unusable.
Fix by zeroing the device info, so all fields default to 0.
Fixes 6c53694fb222 ("Input: synaptics - split device info into a separate structure")
Signed-off-by Eric Biggers <[email protected]>
--- drivers/input/mouse/synaptics.c | 2 ++ 1 file changed, 2 insertions(+)
drivers/input/mouse/synaptics.c +2 -0
@@ -397,6 +397,8 @@ static int synaptics_query_hardware(struct psmouse *psmouse, {int error; +memset (info,0 ,sizeof (*info)); + error =synaptics_identify (psmouse, info);if (error)return error;
Signature · 1 line
2.13.0
Quoted 11 lines · Eric Biggers
From: Eric Biggers <[email protected]>
synaptics_query_hardware()was being passed a
'structsynaptics_device_info' in uninitialized stack memory, then not always initializing all fields. This caused garbage to show up in certain fields, making the touchpad unusable.Fix by zeroing the device info, so all fields default to 0.
Fixes 6c53694fb222 ("Input: synaptics - split device info into a separate structure")Signed-off-by Eric Biggers <[email protected]>
Applied, thank you.
Quoted 20 lines
--- drivers/input/mouse/synaptics.c | 2 ++ 1 file changed, 2 insertions(+)drivers/input/mouse/synaptics.c +2
-0@@ -397,6 +397,8 @@ static int synaptics_query_hardware(struct psmouse *psmouse, {int error; +memset (info,0 ,sizeof (*info)); + error =synaptics_identify (psmouse, info);if (error)return error;Signature · 1 line
2.13.0
Signature · 1 line
Dmitry