Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a80ea8ae authored by Dmitry Torokhov's avatar Dmitry Torokhov Committed by Greg Kroah-Hartman
Browse files

Input: psmouse - fix OOB access in Elantech protocol 

commit 7b63a88bb62ba2ddf5fcd956be85fe46624628b9 upstream.

The kernel only allocate 5 MT slots; check that transmitted slot ID
falls within the acceptable range.

Link: https://lore.kernel.org/r/ZFnEL91nrT789dbG@google.com


Cc: stable@vger.kernel.org
Signed-off-by: default avatarDmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent c41d5cbf

drivers/input/mouse/elantech.c

+5 −4
Original line number Diff line number Diff line
@@ -590,10 +590,11 @@ static void process_packet_head_v4(struct psmouse *psmouse) 
	struct input_dev *dev = psmouse->dev;

	struct elantech_data *etd = psmouse->private;

	unsigned char *packet = psmouse->packet;

	int id = ((packet[3] & 0xe0) >> 5) - 1;

	int id;

	int pres, traces;



	if (id < 0)

	id = ((packet[3] & 0xe0) >> 5) - 1;

	if (id < 0 || id >= ETP_MAX_FINGERS)

		return;



	etd->mt[id].x = ((packet[1] & 0x0f) << 8) | packet[2];
@@ -623,7 +624,7 @@ static void process_packet_motion_v4(struct psmouse *psmouse) 
	int id, sid;



	id = ((packet[0] & 0xe0) >> 5) - 1;

	if (id < 0)

	if (id < 0 || id >= ETP_MAX_FINGERS)

		return;



	sid = ((packet[3] & 0xe0) >> 5) - 1;
@@ -644,7 +645,7 @@ static void process_packet_motion_v4(struct psmouse *psmouse) 
	input_report_abs(dev, ABS_MT_POSITION_X, etd->mt[id].x);

	input_report_abs(dev, ABS_MT_POSITION_Y, etd->mt[id].y);



	if (sid >= 0) {

	if (sid >= 0 && sid < ETP_MAX_FINGERS) {

		etd->mt[sid].x += delta_x2 * weight;

		etd->mt[sid].y -= delta_y2 * weight;

		input_mt_slot(dev, sid);