Re: [PATCH v3 1/4] cdc-acm: fix possible invalid access when processing notification

2017-03-31 Thread Oliver Neukum
Am Donnerstag, den 30.03.2017, 22:15 +0200 schrieb Tobias Herzog: > Notifications may only be 8 bytes long. Accessing the 9th and > 10th byte of unimplemented/unknown notifications may be insecure. > Also check the length of known notifications before accessing anything > behind the 8th byte. > >

[PATCH v3 1/4] cdc-acm: fix possible invalid access when processing notification

2017-03-30 Thread Tobias Herzog
Notifications may only be 8 bytes long. Accessing the 9th and 10th byte of unimplemented/unknown notifications may be insecure. Also check the length of known notifications before accessing anything behind the 8th byte. Signed-off-by: Tobias Herzog --- drivers/usb/class/cdc-acm.c | 13 +-