Pārlūkot izejas kodu

USB: mxu11x0: fix memory leak in port-probe error path

Fix memory leak in port-probe error path by verifying the interrupt-in
urb before allocating the private data.

Signed-off-by: Johan Hovold <johan@kernel.org>
Johan Hovold 9 gadi atpakaļ
vecāks
revīzija
924eccc73d
1 mainītis faili ar 5 papildinājumiem un 7 dzēšanām
  1. 5 7
      drivers/usb/serial/mxu11x0.c

+ 5 - 7
drivers/usb/serial/mxu11x0.c

@@ -333,7 +333,11 @@ static int mxu1_port_probe(struct usb_serial_port *port)
 {
 	struct mxu1_port *mxport;
 	struct mxu1_device *mxdev;
-	struct urb *urb;
+
+	if (!port->interrupt_in_urb) {
+		dev_err(&port->dev, "no interrupt urb\n");
+		return -ENODEV;
+	}
 
 	mxport = kzalloc(sizeof(struct mxu1_port), GFP_KERNEL);
 	if (!mxport)
@@ -344,12 +348,6 @@ static int mxu1_port_probe(struct usb_serial_port *port)
 
 	mxdev = usb_get_serial_data(port->serial);
 
-	urb = port->interrupt_in_urb;
-	if (!urb) {
-		dev_err(&port->dev, "%s - no interrupt urb\n", __func__);
-		return -EINVAL;
-	}
-
 	switch (mxdev->mxd_model) {
 	case MXU1_1110_PRODUCT_ID:
 	case MXU1_1150_PRODUCT_ID: