Re: [patch]fix oops in cdc-wdm in case of malformed descriptors

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Saturday 04 April 2009, Oliver Neukum wrote:
> 
>         iface = &intf->altsetting[0];
>         ep = &iface->endpoint[0].desc;
> -       if (!usb_endpoint_is_int_in(ep)) {
> +       if (!ep || !usb_endpoint_is_int_in(ep)) {
>                 rv = -EINVAL;
>                 goto err;

Clearly this does not fix the underlying problem, though.

Why is it assuming that altsetting has endpoints?
If that interface uses ISO transfers, then the
first altsetting must not have any endpoints...
--
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

[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux