svn commit: r364791 - head/usr.sbin/jail
James Gritton
jamie at freebsd.org
Wed Aug 26 16:34:56 UTC 2020
On 2020-08-26 09:32, Kyle Evans wrote:
> On Wed, Aug 26, 2020 at 6:58 AM Roman Bogorodskiy <novel at freebsd.org>
> wrote:
>>
>> Jamie Gritton wrote:
>>
>> > Author: jamie
>> > Date: Wed Aug 26 00:42:59 2020
>> > New Revision: 364791
>> > URL: https://svnweb.freebsd.org/changeset/base/364791
>> >
>> > Log:
>> > Handle jail.conf variables that have the same names as parameters.
>> >
>> > PR: 248444
>> > Submitted by: Akos Somfai
>> > Reported by: Markus Stoff
>> >
>> > Modified:
>> > head/usr.sbin/jail/config.c
>> >
>> > Modified: head/usr.sbin/jail/config.c
>> > ==============================================================================
>> > --- head/usr.sbin/jail/config.c Wed Aug 26 00:31:59 2020 (r364790)
>> > +++ head/usr.sbin/jail/config.c Wed Aug 26 00:42:59 2020 (r364791)
>> > @@ -393,7 +393,8 @@ add_param(struct cfjail *j, const struct cfparam *p, e
>> > else
>> > for (ipnum = IP__NULL + 1; ipnum < IP_NPARAM; ipnum++)
>> > if (!(intparams[ipnum].flags & PF_CONV) &&
>> > - equalopts(name, intparams[ipnum].name)) {
>> > + equalopts(name, intparams[ipnum].name) &&
>> > + !(p->flags & PF_VAR)) {
>> > j->intparams[ipnum] = np;
>> > np->flags |= intparams[ipnum].flags;
>> > break;
>>
>> Looks like it's causing jail(8) to segfault, at least when using with
>> poudriere:
>>
>
> It looks like it's also wiped out a good chunk of the test suite, as
> reported by lwhsu:
> https://ci.freebsd.org/job/FreeBSD-head-amd64-test/16305/testReport/
Ooh - I killed things nicely, didn't I? I'll get on that.
- Jamie
More information about the svn-src-all
mailing list