On Fri, Sep 03, 2004 at 12:32:07AM +1200, Mike Beattie wrote:
> +       if (strchr(auth_identity, '.') || strlen(auth_identity)+6 >= 
> sizeof(inboxname)) return 0;
...
> +       if (strchr(auth_identity, '.') || strlen(auth_identity)+6 >= 
> sizeof(inboxname)) return 0;

It occurred to me that I'd not found something better to do above, rather
than return 0. This particular line was copied verbatim from acl_ok() from
imapd.c (mainly because config.c had it's own acl_ok(), or I would have
just called the function instead of mucking around creating inboxname for
mboxlist_lookup() ).

I guess something better here would probably be SASL_BADAUTH ?

Mike.
-- 
Mike Beattie  <[EMAIL PROTECTED]>     UNIX Systems Engineer, ITS
Ph: +64 3 479 8597       Fax: +64 3 479 5080      Cell: +64 27 44 80386
* Opinions expressed are my own, not those of the University of Otago *
---
Cyrus Home Page: http://asg.web.cmu.edu/cyrus
Cyrus Wiki/FAQ: http://cyruswiki.andrew.cmu.edu
List Archives/Info: http://asg.web.cmu.edu/cyrus/mailing-list.html

Reply via email to