USB: ljca: drop redundant interface reference

Driver core holds a reference to the USB interface while it is bound to
a driver and there is no need to take another reference unless the
interface is needed after disconnect.

Drop the redundant interface reference to reduce cargo culting, make it
easier to spot drivers where an extra reference is needed, and reduce
the risk of memory leaks when drivers fail to release it.

Signed-off-by: Johan Hovold <johan@kernel.org>
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Link: https://patch.msgid.link/20260305111511.18386-4-johan@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/usb/misc/usb-ljca.c b/drivers/usb/misc/usb-ljca.c
index 7e85fd12da56415ec59909675d751dd677d16421..c60121faa3dae489286caba31978327ad970ed03 100644 (file)
--- a/drivers/usb/misc/usb-ljca.c
+++ b/drivers/usb/misc/usb-ljca.c
@@ -776,7 +776,7 @@ static int ljca_probe(struct usb_interface *interface,
        init_completion(&adap->cmd_completion);
        INIT_LIST_HEAD(&adap->client_list);
 
-       adap->intf = usb_get_intf(interface);
+       adap->intf = interface;
        adap->usb_dev = usb_dev;
        adap->dev = dev;
 
@@ -787,7 +787,7 @@ static int ljca_probe(struct usb_interface *interface,
        ret = usb_find_common_endpoints(alt, &ep_in, &ep_out, NULL, NULL);
        if (ret) {
                dev_err(dev, "bulk endpoints not found\n");
-               goto err_put;
+               goto err_destroy_mutex;
        }
        adap->rx_pipe = usb_rcvbulkpipe(usb_dev, usb_endpoint_num(ep_in));
        adap->tx_pipe = usb_sndbulkpipe(usb_dev, usb_endpoint_num(ep_out));
@@ -797,14 +797,14 @@ static int ljca_probe(struct usb_interface *interface,
        adap->rx_buf = devm_kzalloc(dev, adap->rx_len, GFP_KERNEL);
        if (!adap->rx_buf) {
                ret = -ENOMEM;
-               goto err_put;
+               goto err_destroy_mutex;
        }
 
        /* alloc rx urb */
        adap->rx_urb = usb_alloc_urb(0, GFP_KERNEL);
        if (!adap->rx_urb) {
                ret = -ENOMEM;
-               goto err_put;
+               goto err_destroy_mutex;
        }
        usb_fill_bulk_urb(adap->rx_urb, usb_dev, adap->rx_pipe,
                          adap->rx_buf, adap->rx_len, ljca_recv, adap);
@@ -836,10 +836,7 @@ static int ljca_probe(struct usb_interface *interface,
 
 err_free:
        usb_free_urb(adap->rx_urb);
-
-err_put:
-       usb_put_intf(adap->intf);
-
+err_destroy_mutex:
        mutex_destroy(&adap->mutex);
 
        return ret;
@@ -864,8 +861,6 @@ static void ljca_disconnect(struct usb_interface *interface)
 
        usb_free_urb(adap->rx_urb);
 
-       usb_put_intf(adap->intf);
-
        mutex_destroy(&adap->mutex);
 }