The following reply was made to PR kern/126368; it has been noted by GNATS.

From: "Mateusz Guzik" <[EMAIL PROTECTED]>
To: [EMAIL PROTECTED]
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;
 +      }
        /*
         * 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.
         */
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-jail
To unsubscribe, send any mail to "[EMAIL PROTECTED]"

Reply via email to