Something went wrong -- this isn't exactly the patch I sent (at least not as recorded in my sent-mail). I'm not sure exactly what went wrong, but the 'arg2 &&' should be added about ten lines below where this occurs -- this introduces an error where providing only one argument produces an error message (and nothing is set). (The person who submited the bug applied the patch by hand, as this patch would do, and got that error -- could this patch have come from him? The line numbers are right, from the source code file he showed me...)
I don't know how to get a current snapshot, so I'll get one early tomorrow morning and write a patch against it. Alternately, I've included my patch at the end of this message, in case it's easier to roll back one and apply the other. There are small but significant differences. The line numbers are a bit screwy, though, I guess because my patch was against slightly older source... -- Ed Korthof | Web Server Engineer -- -- [EMAIL PROTECTED] | Organic Online, Inc -- -- (415) 278-5676 | Fax: (415) 284-6891 -- On Sun, 2 Feb 1997, Brian Behlendorf wrote: > brian 97/02/02 18:08:18 > > Modified: src http_core.c > Log: > Reviewed by: Dean Gaudet <[EMAIL PROTECTED]>, Jim Jagielski <[EMAIL > PROTECTED]> > Submitted by: Ed Korthof <[EMAIL PROTECTED]> > > Fix core dump problems with Rlimit directives > > Revision Changes Path > 1.63 +1 -1 apache/src/http_core.c > > Index: http_core.c > =================================================================== > RCS file: /export/home/cvs/apache/src/http_core.c,v > retrieving revision 1.62 > retrieving revision 1.63 > diff -C3 -r1.62 -r1.63 > *** http_core.c 1997/02/01 22:03:36 1.62 > --- http_core.c 1997/02/03 02:08:16 1.63 > *************** > *** 1004,1010 **** > return; > } > > ! if ((str = getword_conf(cmd->pool, &arg))) > if (!strcasecmp(str, "max")) > cur = limit->rlim_max; > else > --- 1004,1010 ---- > return; > } > > ! if (arg2 && (str = getword_conf(cmd->pool, &arg))) > if (!strcasecmp(str, "max")) > cur = limit->rlim_max; > else > > > > *** http_core.c.orig Wed Jan 29 21:34:44 1997 --- http_core.c Wed Jan 29 21:34:17 1997 *************** *** 1003,1009 **** return; } ! if ((str = getword_conf(cmd->pool, &arg2))) max = atol(str); /* if we aren't running as root, cannot increase max */ --- 1003,1009 ---- return; } ! if (arg2 && (str = getword_conf(cmd->pool, &arg2))) max = atol(str); /* if we aren't running as root, cannot increase max */