francis picabia:
[ Charset ISO-8859-1 unsupported, converting... ]
> On Tue, Oct 2, 2012 at 9:20 PM, Wietse Venema <wie...@porcupine.org> wrote:
>
> > Nope. If you were testing this more carefully then you would have
> > found that upper or lower case does not matter in this context.
>
> I tested the exact same line with PERMIT and permit.
>
> permit allowed the whitelist entry to work.
> PERMIT generates a warning.
Found it.
The comparison is case-insensitive except for the first character.
Thus, pERMIT will work, but of course that is not what we want.
Wietse
*** ./src/global/server_acl.c- Fri Dec 23 21:13:31 2011
--- ./src/global/server_acl.c Wed Oct 3 10:04:58 2012
***************
*** 112,119 ****
char *bp = saved_acl;
char *acl;
! #define STREQ(x,y) ((*x) == (*y) && strcasecmp((x), (y)) == 0)
! #define STRNE(x,y) ((*x) != (*y) || strcasecmp((x), (y)) != 0)
/*
* Nested tables are not allowed. Tables are opened before entering the
--- 112,119 ----
char *bp = saved_acl;
char *acl;
! #define STREQ(x,y) (strcasecmp((x), (y)) == 0)
! #define STRNE(x,y) (strcasecmp((x), (y)) != 0)
/*
* Nested tables are not allowed. Tables are opened before entering the
