Re: [PATCH] usb: kfree() cleanups in drivers/usb/core/devio.c

[Jesper Juhl]

On Tue, 12 Apr 2005, Greg KH wrote:

> Date: Tue, 12 Apr 2005 00:40:56 -0700
> From: Greg KH <[EMAIL PROTECTED]>
> To: Jesper Juhl <[EMAIL PROTECTED]>
> Cc: linux-kernel@vger.kernel.org, Thomas Sailer <[EMAIL PROTECTED]>,
> Greg Kroah-Hartman <[EMAIL PROTECTED]>, 
> linux-usb-devel@lists.sourceforge.net
> Subject: Re: [PATCH] usb: kfree() cleanups in drivers/usb/core/devio.c
> 
> On Mon, Apr 11, 2005 at 11:55:22PM +0200, Jesper Juhl wrote:
> > Checking for NULL before calling kfree() is redundant. This patch removes 
> > these redundant checks and also makes a few tiny whitespace changes.
> > 
> > Signed-off-by: Jesper Juhl <[EMAIL PROTECTED]>
> 
> Applied, thanks.
> 
You're welcome. I have a patch 90% done that makes the same change for all 
of drivers/usb/* want me to send that along or would you prefer I stick to 
just drivers/usb/core/* ?  One huge patch OK or would you prefer it split 
into one patch pr modified file?   
I can send the patch later tonight when I get home from work.

-- 
Jesper Juhl

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] usb: kfree() cleanups in drivers/usb/core/devio.c

[Greg KH]

On Mon, Apr 11, 2005 at 11:55:22PM +0200, Jesper Juhl wrote:
> Checking for NULL before calling kfree() is redundant. This patch removes 
> these redundant checks and also makes a few tiny whitespace changes.
> 
> Signed-off-by: Jesper Juhl <[EMAIL PROTECTED]>

Applied, thanks.

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH] usb: kfree() cleanups in drivers/usb/core/devio.c

[Jesper Juhl]

Checking for NULL before calling kfree() is redundant. This patch removes 
these redundant checks and also makes a few tiny whitespace changes.

Signed-off-by: Jesper Juhl <[EMAIL PROTECTED]>
---

 devio.c |   32 
 1 files changed, 12 insertions(+), 20 deletions(-)

--- linux-2.6.12-rc2-mm3-orig/drivers/usb/core/devio.c  2005-04-11 
21:20:49.0 +0200
+++ linux-2.6.12-rc2-mm3/drivers/usb/core/devio.c   2005-04-11 
23:49:41.0 +0200
@@ -213,12 +213,10 @@ static struct async *alloc_async(unsigne
 
 static void free_async(struct async *as)
 {
-if (as->urb->transfer_buffer)
-kfree(as->urb->transfer_buffer);
-if (as->urb->setup_packet)
-kfree(as->urb->setup_packet);
+   kfree(as->urb->transfer_buffer);
+   kfree(as->urb->setup_packet);
usb_free_urb(as->urb);
-kfree(as);
+   kfree(as);
 }
 
 static inline void async_newpending(struct async *as)
@@ -938,17 +936,13 @@ static int proc_do_submiturb(struct dev_
return -EINVAL;
}
if (!(as = alloc_async(uurb->number_of_packets))) {
-   if (isopkt)
-   kfree(isopkt);
-   if (dr)
-   kfree(dr);
+   kfree(isopkt);
+   kfree(dr);
return -ENOMEM;
}
if (!(as->urb->transfer_buffer = kmalloc(uurb->buffer_length, 
GFP_KERNEL))) {
-   if (isopkt)
-   kfree(isopkt);
-   if (dr)
-   kfree(dr);
+   kfree(isopkt);
+   kfree(dr);
free_async(as);
return -ENOMEM;
}
@@ -967,8 +961,7 @@ static int proc_do_submiturb(struct dev_
as->urb->iso_frame_desc[u].length = isopkt[u].length;
totlen += isopkt[u].length;
}
-   if (isopkt)
-   kfree(isopkt);
+   kfree(isopkt);
as->ps = ps;
 as->userurb = arg;
if (uurb->endpoint & USB_DIR_IN)
@@ -1237,7 +1230,7 @@ static int proc_ioctl (struct dev_state 
return -ENOMEM;
if ((_IOC_DIR(ctrl.ioctl_code) & _IOC_WRITE)) {
if (copy_from_user (buf, ctrl.data, size)) {
-   kfree (buf);
+   kfree(buf);
return -EFAULT;
}
} else {
@@ -1246,8 +1239,7 @@ static int proc_ioctl (struct dev_state 
}
 
if (!connected(ps->dev)) {
-   if (buf)
-   kfree(buf);
+   kfree(buf);
return -ENODEV;
}
 
@@ -1309,8 +1301,8 @@ static int proc_ioctl (struct dev_state 
&& size > 0
&& copy_to_user (ctrl.data, buf, size) != 0)
retval = -EFAULT;
-   if (buf != NULL)
-   kfree (buf);
+
+   kfree(buf);
return retval;
 }
 



-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/