Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 20 Oct 2002 13:59:17 -0700
From:      Terry Lambert <tlambert2@mindspring.com>
To:        "M. Warner Losh" <imp@bsdimp.com>
Cc:        jake@locore.ca, ben@stuyts.nl, current@FreeBSD.ORG, jroberson@chesapeake.net, rwatson@FreeBSD.ORG, jeff@FreeBSD.ORG, alfred@FreeBSD.ORG
Subject:   Re: [Ugly PATCH] Again: panic kmem_malloc()
Message-ID:  <3DB31925.7B70E70@mindspring.com>
References:  <3DB07805.AB4BAA15@mindspring.com> <4.3.2.7.2.20021019001010.00b89f28@terminus> <20021018213608.A60569@locore.ca> <20021020.125308.35255652.imp@bsdimp.com>

next in thread | previous in thread | raw e-mail | index | archive | help
"M. Warner Losh" wrote:
> : +     if (sops)
> : +             free(sops, M_SEM);
> 
> The kernel free() groks free(NULL, M_FOO), so the if isn't needed.

Wow.  That's bogus.  It should panic.

Or we should fix all of libc to take NULL arguments for strings,
and treat them as if they were actually "".

-- Terry

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3DB31925.7B70E70>