im.pidgin.pidgin: 3cec6952fcadbd73dddc4c009e798b6fb1789a5b
Sadrul Habib Chowdhury
imadil at gmail.com
Sat Dec 22 13:51:51 EST 2007
* Evan Schoenberg had this to say on [22 Dec 2007, 10:45:41 -0500]:
>
> On Dec 21, 2007, at 3:44 PM, Daniel Atallah wrote:
>
>> Added support for authentication via CRAM-MD5 when using jabber:iq:auth.
>> This, combined with 33785a8c844de496aeeed927bad8b39a6e091931, fixes
>> connectivity with iChat Server 10.5 (a jabberd derivative with SASL
>> support), among other servers, when libpurple is compiled with SASL
>> support.
>>
>>
>> This looks not quite right to me - the fifth argument to auth_hmac_md5()
>> is a guchar*, but it is being passed a guchar**.
>
> I agree - that's not right. I find it very strange that it both works as I
> committed it originally (wrong type) and as I've just committed in
> 16e6cd4ffd8a8308380dc016f0afa782a7750374 (Just pass 'digest', not a pointer
> to digest)... Magic? Thanks for the catch... and if there's learning to be
> had as to why both versions worked, I'd love to hear it :)
>
For 'char digest[...];', all of 'digest', '&digest' and '&digest[0]' are
equivalent. I think that's why both of the versions work.
Sadrul
More information about the Devel
mailing list