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).
Is there a good reason for this? POSIX requires that free() be NULL-safe. This may not apply to the kernel's free(), but it seems like something that programmers would expect.
