From: George Kennedy <george.kenn...@oracle.com> Date: Tue, 14 Jul 2020 17:34:33 -0400
> For example, the failing case here has "ret" = 0 (#define ETH_ALEN 6): > > 172 static int ax88172a_bind(struct usbnet *dev, struct > usb_interface *intf) > 173 { > ... > 186 /* Get the MAC address */ > 187 ret = asix_read_cmd(dev, AX_CMD_READ_NODE_ID, 0, 0, > ETH_ALEN, buf, 0); > 188 if (ret < ETH_ALEN) { > 189 netdev_err(dev->net, "Failed to read MAC > address: %d\n", ret); > 190 goto free; > 191 } > "drivers/net/usb/ax88172a.c" Then this is the spot that should set 'ret' to -EINVAL or similar?