svn commit: r230125 - head/sys/kern

John Baldwin jhb at freebsd.org
Tue Jan 17 18:48:11 UTC 2012


On Tuesday, January 17, 2012 12:29:06 pm Andriy Gapon wrote:
> on 17/01/2012 19:12 Eitan Adler said the following:
> > To me seeing "(null)" from the output of printf indicates "There is a
> > bug here, printf should never be given a NULL argument.".
> > However, given that it isn't undefined here, and the objections I've
> > received I will revert this part of the change when I get home.
> 
> Would this be worth the hassle now that the change is already committed?

Probably not, though at some point if the printf is reworked for some other
reason it could be removed then.  I don't think we need to add more special
case handling for NULL string pointers passed to kernel printf in the future,
however.

-- 
John Baldwin


More information about the svn-src-head mailing list