Cyrus IMAPd 2.3.10 Released

[John Capo]

On Thu, October 25, 2007 21:10, John Capo wrote:
> Quoting Ken Murchison (murch at andrew.cmu.edu):
>
>> Simon Matter wrote:
>>
>>>> On the Linux box, all fresh compilations aside from the sasl 2.1.15 binaries:
>>>>
>>>
>>> I once posted to the list that 2.3.9 needs at least cyrus-sasl-2.1.19. As a package
>>> maintainer I know that :)
>>
>> Did you ever figure out why?  I'm not surprised that code in Cyrus somehow depends on
>> a change in SASL, but I can't seem to find anything in the CVS logs or diffs that
>> would be the cause.
>
> This is what I had to do for cmd_login to work in 2.3.9.
>
>
> /* authstate already created by mysasl_proxy_policy() */
> /* Not when using login and allowplaintext.  imapd_authstate is NULL  TM Login fix */
> if (imapd_authstate == NULL)
>     imapd_authstate = auth_newstate(imapd_userid);
>
> But 2.3.10 cores :-(

Its coring in getgrouplist() probably because the 3rd argyument is NULL.

    /* get number of groups user is member of into ngroups */
    getgrouplist(identifier, gid, NULL, &ngroups);

BSD man page does not indicate that NULL args are OK.

  int
  getgrouplist(const char *name, int basegid, int *groups, int *ngroups);

 The resulting group list is returned in the integer array pointed to by
 groups.  The caller specifies the size of the groups array in the integer
 pointed to by ngroups; the actual number of groups found is returned in
 ngroups.

A non NULL imapd_authstate and turing off unix_group_enable works with older SASL
libraries and 2.3.10.



>
>
>
>
>>
>>
>> -- Kenneth Murchison Systems Programmer Project Cyrus Developer/Maintainer Carnegie
>> Mellon University ---- Cyrus Home Page: http://cyrusimap.web.cmu.edu/ Cyrus Wiki/FAQ:
>> http://cyrusimap.web.cmu.edu/twiki List Archives/Info:
>> http://asg.web.cmu.edu/cyrus/mailing-list.html
>>
>