Re: [PATCH v2] cdc-acm: prevent infinite loop when parsing CDC headers.

2015-04-20 Thread Quentin Casasnovas
On Tue, Apr 14, 2015 at 11:25:43AM +0200, Quentin Casasnovas wrote: Phil and I found out a problem with commit: 7e860a6e7aa6 (cdc-acm: add sanity checks) Any comment on v2? Thanks, Quentin -- To unsubscribe from this list: send the line unsubscribe linux-usb in the body of a message to

Re: [PATCH v2] cdc-acm: prevent infinite loop when parsing CDC headers.

2015-04-20 Thread Greg Kroah-Hartman
On Mon, Apr 20, 2015 at 01:54:56PM +0200, Quentin Casasnovas wrote: On Tue, Apr 14, 2015 at 11:25:43AM +0200, Quentin Casasnovas wrote: Phil and I found out a problem with commit: 7e860a6e7aa6 (cdc-acm: add sanity checks) Any comment on v2? I can't do anything with any patches

[PATCH v2] cdc-acm: prevent infinite loop when parsing CDC headers.

2015-04-14 Thread Quentin Casasnovas
Phil and I found out a problem with commit: 7e860a6e7aa6 (cdc-acm: add sanity checks) It added some sanity checks to ignore potential garbage in CDC headers but also introduced a potential infinite loop. This can happen at the first loop iteration (elength = 0 in that case) if the description