Date: Fri, 8 Aug 2008 18:43:38 +0000 (UTC) From: "Bjoern A. Zeeb" <bz@FreeBSD.org> To: Mateusz Guzik <mjguzik@gmail.com> Cc: freebsd-jail@FreeBSD.org Subject: Re: kern/126368: Running ktrace/kdump in jail leads to stale jails Message-ID: <20080808184224.H88849@maildrop.int.zabbadoz.net> In-Reply-To: <200808081740.m78He4bc084276@freefall.freebsd.org> References: <200808081740.m78He4bc084276@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, 8 Aug 2008, Mateusz Guzik wrote: > The following reply was made to PR kern/126368; it has been noted by GNATS. > > From: "Mateusz Guzik" <mjguzik@gmail.com> > To: bug-followup@freebsd.org > Cc: > Subject: Re: kern/126368: Running ktrace/kdump in jail leads to stale jails > Date: Fri, 8 Aug 2008 19:30:22 +0200 > > Err, I made a mistake. crfree() will be called in case of failure > (loop starting at line 959), so the following patch should be ok: > > --- sys/kern/kern_ktrace.c.orig 2008-08-08 16:37:45.000000000 +0200 > +++ sys/kern/kern_ktrace.c 2008-08-08 19:25:16.000000000 +0200 > @@ -933,12 +933,14 @@ > error = VOP_WRITE(vp, &auio, IO_UNIT | IO_APPEND, cred); > VOP_UNLOCK(vp, 0, td); > vn_finished_write(mp); > vrele(vp); > VFS_UNLOCK_GIANT(vfslocked); > - if (!error) > + if (!error) { > + crfree(cred); > return; > + } that sounds more plausible w/o seeing the surrounding code. I had wondered already earlier today when I was pointed at. I'll look into this. > /* > * If error encountered, give up tracing on this vnode. We defer > * all the vrele()'s on the vnode until after we are finished walking > * the various lists to avoid needlessly holding locks. > */ > _______________________________________________ > freebsd-jail@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-jail > To unsubscribe, send any mail to "freebsd-jail-unsubscribe@freebsd.org" > -- Bjoern A. Zeeb Stop bit received. Insert coin for new game.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080808184224.H88849>