On Wed, Aug 28, 2002 at 08:30:41AM -0400, Kev wrote:
> > If no target is specified in /gline command for opers a local gline is
> > still added evne if FEAT_LGLINE is false. patch by darren coleman.
>

Was actually Darren Smith, but no harm no foul :)


> The patch you sent wouldn't have worked quite the way you intended--it
> would require you to have the PRIV_LOCAL_GLINE privilege to set global
> G-lines.  While it could be argued that you should have both to set
> global G-lines, it does break the separation of privilege I was intending.
> Of course, fixing this was easy--I just replaced the "PRIV_LOCAL_GLINE"
> with a ?: operator testing on whether the GLINE_LOCAL flag got set.  I
> also removed the now redundant tests from where the target is checked.
> While I was in there, I also noted another burr having to do with
> wide glines and the WIDE_GLINE privilege, which I corrected.
>
> Thanks for pointing this problem out!
> --
> Kevin L. Mitchell <[EMAIL PROTECTED]>
>


Cheers for fixing this kev, saved us a patch :)
-- 
Joe "Pilsner" Cise                (__)         e::[EMAIL PROTECTED]
Co-Founder, b0rk UK                oo\\\~      w::www.b0rk.co.uk
Senior IRC Admin, QuakeNet          !!!!       w::www.quakenet.org

Reply via email to