> Simon Matter wrote:
>>> Hi,
>>>
>>> I'm in the process of upgrading to 2.3.14 and looking at the gcc
>>> warnings resulted in this change:
>>>
>>> diff --git a/imap/lmtpengine.c b/imap/lmtpengine.c
>>> index 3df2911..36d53bc 100644
>>> --- a/imap/lmtpengine.c
>>> +++ b/imap/lmtpengine.c
>>> @@ -802,7 +802,7 @@ static int savemsg(struct clientdata *cd,
>>> static int process_recipient(char *addr, struct namespace *namespace,
>>> int ignorequota,
>>> int (*verify_user)(const char *, const char *,
>>> - char *, long,
>>> + char *, quota_t,
>>> struct auth_state *),
>>> message_data_t *msg)
>>> {
>>>
>>> IMHO this can very well explain the difference between 32-bit and
>>> 64-bit
>>> behaviour, when sizeof(quota_t) != sizeof(long). I've not tested it yet
>>> though.
>>
>> Hi Gabor,
>>
>> Thanks for looking at the GCC warning! I have verified it and that's
>> exactly where it fails.
>> That really explains why not all platforms were affected.
>
> I believe if you look at the cvs logs, this has already been patched.
> Sorry I didn't think to mention this earlier in the thread.
Thanks, now I see it.
Gabor, you may have a look at the patch here:
http://github.com/brong/cyrus-imapd/commit/ec1bfcf6a1db9c86cbf55b9c25d7eb044dbbe51b#diff-0
Simon
----
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
