[PATCH] Remove more superfluous if(p!=NULL) checks around free(p).

Mark Kettenis mark.kettenis at xs4all.nl
Wed Nov 10 06:57:42 PST 2010


> X-Original-To: xorg-devel at lists.x.org
> X-Greylist: delayed 302 seconds by postgrey-1.31 at gabe;
> 	Wed, 10 Nov 2010 06:29:10 PST
> From: Cyril Brulebois <kibi at debian.org>
> Date: Wed, 10 Nov 2010 15:28:29 +0100
> Sender: xorg-devel-bounces+mark.kettenis=xs4all.nl at lists.x.org
> X-XS4ALL-DNSBL-Checked: mxdrop216.xs4all.nl checked 131.252.210.177 against DNS blacklists
> X-CNFS-Analysis: v=1.1 cv=a8sYchbnJd2dMYUqqjolUMD0rF/qLqJCTuzWyWz0xZo= c=1
>  sm=0 a=kNH2u6Mz06QA:10 a=kj9zAlcOel0A:10 a=xqWC_Br6kY4A:10
>  a=NZLgQZmgF9XIoAvTQ72Ilw==:17 a=xNf9USuDAAAA:8 a=aR16PxjQAAAA:8
>  a=YT3-SkiptP_M-PE1JGAA:9 a=1IQqWsoUNsSykDe4CdVREsF8W8MA:4 a=CjuIK1q_8ugA:10
>  a=CiSHi91Bn78A:10 a=YPTUPuSgPjgA:10 a=NZLgQZmgF9XIoAvTQ72Ilw==:117
> X-Virus-Scanned: by XS4ALL Virus Scanner
> X-XS4ALL-Spam-Score: 0.0 () none
> X-XS4ALL-Spam: NO
> Envelope-To: mark.kettenis at xs4all.nl
> 
> This patch has been generated by the following Coccinelle semantic patch:
> 
> @@
> expression E;
> @@
> - if (E != NULL) {
> -   free(E);
> - }
> + free(E);
> 
> Signed-off-by: Cyril Brulebois <kibi at debian.org>
> ---
>  glx/glxdri.c           |    4 +---
>  glx/glxdri2.c          |    4 +---
>  glx/single2.c          |    4 +---
>  hw/xwin/glx/indirect.c |    5 +----
>  4 files changed, 4 insertions(+), 13 deletions(-)

Reviewed-by: Mark Kettenis <kettenis at openbsd.org>


More information about the xorg-devel mailing list