From owner-freebsd-bugs Tue Jan 21 08:01:07 1997 Return-Path: Received: (from root@localhost) by freefall.freebsd.org (8.8.4/8.8.4) id IAA26417 for bugs-outgoing; Tue, 21 Jan 1997 08:01:07 -0800 (PST) Received: from godzilla.zeta.org.au (godzilla.zeta.org.au [203.2.228.19]) by freefall.freebsd.org (8.8.4/8.8.4) with ESMTP id IAA26412 for ; Tue, 21 Jan 1997 08:01:03 -0800 (PST) Received: (from bde@localhost) by godzilla.zeta.org.au (8.8.3/8.6.9) id CAA10696; Wed, 22 Jan 1997 02:58:56 +1100 Date: Wed, 22 Jan 1997 02:58:56 +1100 From: Bruce Evans Message-Id: <199701211558.CAA10696@godzilla.zeta.org.au> To: davidn@unique.usn.blaze.net.au Subject: Re: kern/2535: filesize-cur resource limit reset to "infinity" Cc: freebsd-bugs@freefall.freebsd.org Sender: owner-bugs@FreeBSD.ORG X-Loop: FreeBSD.org Precedence: bulk >> Is this patch the correct fix? >> >> --- kern_exit.c.orig Wed Jan 15 14:55:26 1997 >> +++ kern_exit.c Tue Jan 21 12:48:57 1997 >> @@ -222,6 +222,10 @@ >> sp->s_leader = NULL; >> } >> fixjobc(p, p->p_pgrp, 0); >> + if (p->p_limit->p_refcnt > 1 && p->p_limit->p_lflags & PL_SHAREMOD) { > >Whoops. That should of course be: > >> + if (p->p_limit->p_refcnt > 1 && (p->p_limit->p_lflags & PL_SHAREMOD) == 0) { > Except it is has too many columns. See kern_resource.c for correct formatting. BTW, PL_SHAREMOD seems to be never set. It is to support threads, but the kernel doesn't have any real support for threads. Bruce