Re: [Qemu-devel] [PATCH v2 4/8] usb-ccid: Fix ATR parsing

2017-07-20 Thread Marc-André Lureau
On Thu, Jul 20, 2017 at 11:05 AM Stefan Fritsch wrote: > From: Stefan Fritsch > > - The number of parameter set TA_i...TD_i is unlimited. The list ends > if TD_i is not present or the high nibble is zero. > - If at least one protocol in any of the TD bytes is non-zero, the > ATR must have a

[Qemu-devel] [PATCH v2 4/8] usb-ccid: Fix ATR parsing

2017-07-20 Thread Stefan Fritsch
From: Stefan Fritsch - The number of parameter set TA_i...TD_i is unlimited. The list ends if TD_i is not present or the high nibble is zero. - If at least one protocol in any of the TD bytes is non-zero, the ATR must have a checksum. - Add a missing length check before accessing TD. - Fixup