lkml.org 
[lkml]   [2005]   [Apr]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] parisc: kfree cleanup in arch/parisc/
On Mon, Apr 11, 2005 at 10:54:25PM +0200, Jesper Juhl wrote:
> Get rid of redundant NULL pointer checks before kfree() in arch/parisc/ as
> well as a few blank lines.
>
> Signed-off-by: Jesper Juhl <juhl-lkml@dif.dk>

thanks!
Commited to cvs.parisc-linux.org:
http://lists.parisc-linux.org/pipermail/parisc-linux-cvs/2005-April/035542.html

I expect Matthew Wilcox will submit to linus with other parisc code changes
in the next RC release.

thanks,
grant

>
> diff -upr linux-2.6.12-rc2-mm3-orig/arch/parisc/kernel/ioctl32.c linux-2.6.12-rc2-mm3/arch/parisc/kernel/ioctl32.c
> --- linux-2.6.12-rc2-mm3-orig/arch/parisc/kernel/ioctl32.c 2005-04-05 21:21:08.000000000 +0200
> +++ linux-2.6.12-rc2-mm3/arch/parisc/kernel/ioctl32.c 2005-04-11 22:48:03.000000000 +0200
> @@ -104,12 +104,9 @@ static int drm32_version(unsigned int fd
> }
>
> out:
> - if (kversion.name)
> - kfree(kversion.name);
> - if (kversion.date)
> - kfree(kversion.date);
> - if (kversion.desc)
> - kfree(kversion.desc);
> + kfree(kversion.name);
> + kfree(kversion.date);
> + kfree(kversion.desc);
> return ret;
> }
>
> @@ -166,9 +163,7 @@ static int drm32_getsetunique(unsigned i
> ret = -EFAULT;
> }
>
> - if (karg.unique != NULL)
> - kfree(karg.unique);
> -
> + kfree(karg.unique);
> return ret;
> }
>
> @@ -265,7 +260,6 @@ static int drm32_info_bufs(unsigned int
> }
>
> kfree(karg.list);
> -
> return ret;
> }
>
> @@ -305,7 +299,6 @@ static int drm32_free_bufs(unsigned int
>
> out:
> kfree(karg.list);
> -
> return ret;
> }
>
> @@ -494,15 +487,10 @@ static int drm32_dma(unsigned int fd, un
> }
>
> out:
> - if (karg.send_indices)
> - kfree(karg.send_indices);
> - if (karg.send_sizes)
> - kfree(karg.send_sizes);
> - if (karg.request_indices)
> - kfree(karg.request_indices);
> - if (karg.request_sizes)
> - kfree(karg.request_sizes);
> -
> + kfree(karg.send_indices);
> + kfree(karg.send_sizes);
> + kfree(karg.request_indices);
> + kfree(karg.request_sizes);
> return ret;
> }
>
> @@ -555,9 +543,7 @@ static int drm32_res_ctx(unsigned int fd
> ret = -EFAULT;
> }
>
> - if (karg.contexts)
> - kfree(karg.contexts);
> -
> + kfree(karg.contexts);
> return ret;
> }
>
>
>
>
>
> PS. If you reply to lists other than Linux-kernel, then please keep me on CC:
>
>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-04-13 00:17    [W:0.236 / U:0.152 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site