Hello,
Johan Hovold 於 2015/3/14 下午 07:48 寫道:
On Thu, Feb 26, 2015 at 06:02:08PM +0800, Peter Hung wrote:
- if (!urb->actual_length)
+ if ((urb->actual_length < 2) || (urb->actual_length % 2))
return;
Not parsing short data (e.g. not divisible by 2) is OK I guess. You
could also just discard the last odd byte, but that's up to you.
Either way, I think you should add a dev_warn here rather than just
silently drop it.
With F81232, when it first submit with interrupt URB, it will response
once with 1 bytes data. The data is hw current LSR, but it useless on
open. It's should necessary with receiving data. When the device is
working good, it's should acked with even size data.
To avoid confusing to user, I'll keep it without warning message.
Thanks
--
With Best Regards,
Peter Hung
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
