On Tue, Sep 15, 2015 at 8:37 PM, Ted Unangst <t...@tedunangst.com> wrote:
> Miod Vallat wrote: > > > Hi, > > > > > > Here a first sets of "if(x) free(x)" cleanup in sys/arch/ > > > > > > This patch contains only trivial if(x) removal. The size argument in > > > free is keep untouched (because it is already setted, or because it > > > makes sens to keep it to 0). > > > > > > Comments ? OK ? > > > > The arch/*/stand use a free() implementation which does not accept NULL > > pointers (see sys/lib/libsa/alloc.c). > > But we're going to fix that too, right? > > It seems that now NULL can be passed to this free implmenation. This has changed yesterday in revision 1.11 ( http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys/lib/libsa/alloc.c.diff?r1=1.10&r2=1.11&f=h ). Regards, Rafael Neves
