Commit 9f6a8dca authored Mar 16, 2017 by Johan Hovold Committed by Greg Kroah-Hartman Mar 30, 2017

Input: yealink - validate number of endpoints before using them



commit 5cc4a1a9f5c179795c8a1f2b0f4361829d6a070e upstream.

Make sure to check the number of endpoints to avoid dereferencing a
NULL-pointer should a malicious device lack endpoints.

Fixes: aca951a2 ("[PATCH] input-driver-yealink-P1K-usb-phone")
Signed-off-by: Johan Hovold <johan@kernel.org>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent d9a3e642

drivers/input/misc/yealink.c

+4 −0

Original line number	Diff line number	Diff line
		@@ -875,6 +875,10 @@ static int usb_probe(struct usb_interface intf, const struct usb_device_id id)
		int ret, pipe, i;

		interface = intf->cur_altsetting;

		if (interface->desc.bNumEndpoints < 1)
		return -ENODEV;

		endpoint = &interface->endpoint[0].desc;
		if (!usb_endpoint_is_int_in(endpoint))
		return -ENODEV;