Re: [PATCH] ppp: ignore unrecognised radiusclient configuration directives

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Is anyone maintaining linux ppp? 2nd repost of this patch.

On Fri, 23 Nov 2012, Charlie Brady wrote:

> 
> Paul, did you just miss this, or are you waiting for some more 
> information? Or do we need David Skoll's input?
> 
> This is relevant for RHEL6, BTW, where default freeradiusclient 
> configuration includes new 'bindaddr' keyword.
> 
> On Thu, 30 Aug 2012, Charlie Brady wrote:
> 
> > New versions of freeradius occasionally add configuration keywords. 
> > There's no reason that the pppd plugin should fail when that occurs.
> > 
> > Signed-off-by:  Charlie Brady <charlieb@xxxxxxxxxxxxxxxxxx>
> > 
> > --- ppp-2.4.5.rhel6/pppd/plugins/radius/config.c        2009-11-16 17:26:07.000000000 -0500
> > +++ ppp-2.4.5/pppd/plugins/radius/config.c      2012-08-30 16:59:19.911025000 -0400
> > @@ -216,8 +216,8 @@
> >                 p[pos] = '\0';
> >  
> >                 if ((option = find_option(p, OT_ANY)) == NULL) {
> > -                       error("%s: line %d: unrecognized keyword: %s", filename, line, p);
> > -                       return (-1);
> > +                       warn("%s: line %d: unrecognized keyword: %s", filename, line, p);
> > +                       continue;
> >                 }
> >  
> >                 if (option->status != ST_UNDEF) {
> > 
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-ppp" in
> > the body of a message to majordomo@xxxxxxxxxxxxxxx
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-ppp" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-ppp" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Audio Users]     [Linux for Hams]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Fedora Users]

  Powered by Linux