cvs commit: src/sys/kern kern_exit.c

John Baldwin jhb at freebsd.org
Tue Oct 24 18:47:38 UTC 2006


On Monday 23 October 2006 18:47, David Xu wrote:
> On Monday 23 October 2006 23:33, John Baldwin wrote:
> > On Saturday 21 October 2006 19:59, David Xu wrote:
> > > davidxu     2006-10-21 23:59:15 UTC
> > >
> > >   FreeBSD src repository
> > >
> > >   Modified files:
> > >     sys/kern             kern_exit.c
> > >   Log:
> > >   Since revision 1.333 of kern_sig.c no longer uses P_WEXIT, the change
> > >   opened a race window which can cause memory leak in signal queue.
> > >   Here we free memory for signal queue when process state is set to
> > >   PRS_ZOMBIE.
> >
> > Is there any reason to not just harvest it in wait() instead?
> 
> I prefer to free it earlier rather than delaying it if I can.

But we already do the free'ing "later" (wait() really isnt' all that later 
than exit1() for most processes) for sigacts, limits, etc.  It would seem 
that sigqueue and sigacts would have nearly identical lifetimes.

-- 
John Baldwin


More information about the cvs-all mailing list