svn commit: r245494 - head/bin/pwait

John Baldwin jhb at freebsd.org
Wed Jan 16 18:43:04 UTC 2013


On Wednesday, January 16, 2013 1:49:40 am Xin LI wrote:
> This doesn't seem right -- you should never release memory before exit,
> especially for memory allocated in main(), unless this "main" is intended
> for different purpose like a monolithic shell that wants to avoid exec().
>  Note that pwait(1) have multiple exit points I don't think it's practical.
> 
> Would you mind if I commit this changeset instead?  I have the return ->
> exit change in my queue long ago but only noticed it today...

I think the free shouldn't be there as well, but I think requiring an exit() 
instead of return to "fix" it is bogus as well.  The static analyzer is just
broken in this case.  main() is special and returns from it should be
treated like exit() and not cause false warnings about memory leaks.

-- 
John Baldwin


More information about the svn-src-all mailing list