This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] vfprintf: validate nargs and maybe allocate from heap

From: Kees Cook <kees at outflux dot net>
To: Andreas Schwab <schwab at linux-m68k dot org>
Cc: Paul Eggert <eggert at cs dot ucla dot edu>, libc-alpha at sourceware dot org
Date: Mon, 6 Feb 2012 09:17:06 -0800
Subject: Re: [PATCH] vfprintf: validate nargs and maybe allocate from heap
References: <20120206062537.GM4979@outflux.net><4F2F7E1C.5010506@cs.ucla.edu><m2mx8w6lcz.fsf@igel.home>

On Mon, Feb 06, 2012 at 04:18:52PM +0100, Andreas Schwab wrote:
> Paul Eggert <eggert@cs.ucla.edu> writes:
> 
> >   all_done:
> >     if (__builtin_expect (args_malloced != NULL, 0))
> >       free (args_malloced);
> 
> No need for the NULL check.

Ah, perhaps I misunderstood earlier style requirements. I was told not to
use implicit NULL tests. What're the other conventions around that?

Thanks,

-Kees

-- 
Kees Cook                                            @outflux.net

Follow-Ups:
- Re: [PATCH] vfprintf: validate nargs and maybe allocate from heap
  - From: Roland McGrath

References:
- [PATCH] vfprintf: validate nargs and maybe allocate from heap
  - From: Kees Cook
- Re: [PATCH] vfprintf: validate nargs and maybe allocate from heap
  - From: Paul Eggert
- Re: [PATCH] vfprintf: validate nargs and maybe allocate from heap
  - From: Andreas Schwab

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]