LDAP auth, and Cyrus mailboxes..

Mike Beattie mike.beattie at otago.ac.nz
Thu Sep 2 15:51:30 EDT 2004


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  <mike.beattie at otago.ac.nz>     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




More information about the Info-cyrus mailing list