Commit 94d7eeb6 authored by Foster Snowhill's avatar Foster Snowhill Committed by David S. Miller

usbnet: ipheth: drop RX URBs with no payload

On iPhone 15 Pro Max one can observe periodic URBs with no payload
on the "bulk in" (RX) endpoint. These don't seem to do anything
meaningful. Reproduced on iOS 17.5.1 and 17.6.

This behaviour isn't observed on iPhone 11 on the same iOS version. The
nature of these zero-length URBs is so far unknown.

Drop RX URBs with no payload.
Signed-off-by: default avatarFoster Snowhill <forst@pen.gy>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 655b46d7
...@@ -286,6 +286,12 @@ static void ipheth_rcvbulk_callback(struct urb *urb) ...@@ -286,6 +286,12 @@ static void ipheth_rcvbulk_callback(struct urb *urb)
return; return;
} }
/* iPhone may periodically send URBs with no payload
* on the "bulk in" endpoint. It is safe to ignore them.
*/
if (urb->actual_length == 0)
goto rx_submit;
/* RX URBs starting with 0x00 0x01 do not encapsulate Ethernet frames, /* RX URBs starting with 0x00 0x01 do not encapsulate Ethernet frames,
* but rather are control frames. Their purpose is not documented, and * but rather are control frames. Their purpose is not documented, and
* they don't affect driver functionality, okay to drop them. * they don't affect driver functionality, okay to drop them.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment