[Kamailio-Devel] [Bulk] Re: [tm] t_lookupOriginalT : cannot find matching INVITE for CANCEL because of display name
Aurelien Grimaud
gstelzz at yahoo.fr
Tue Jan 20 18:57:26 CET 2009
Sorry for false alarm ...
My sipp scenario has a wrong magic cookie !!!
I should have seen it.
> if (branch && branch->value.s && branch->value.len>MCOOKIE_LEN
> && memcmp(branch->value.s,MCOOKIE,MCOOKIE_LEN)==0) {
> /* huhuhu! the cookie is there -- let's proceed fast */
> LOCK_HASH(hash_index);
> ret=matching_3261(p_msg, &p_cell,
> /* we are seeking the original transaction --
> * skip CANCEL transactions during search
> */
> METHOD_CANCEL);
> if (ret==1) goto found; else goto notfound;
> }
Bug is invalid !
Aurelien
Iñaki Baz Castillo a écrit :
> 2009/1/20 Daniel-Constantin Mierla <miconda at gmail.com>:
>
>
>> The first invite (see below) and the cancel has a branch that does not
>> follow the RFC3261 matching rules.
>>
>> rfc3261 states that the cookie must start with "z9hG4bK" in order to use it
>> for matching. Maybe my memories about rfc3261 transaction matching are
>> slowly going away, bit is how I remember it should be done. I can double
>> check it again.
>>
>> So, if I am right (pretty sure about), the branch id is not starting with
>> the magic cookie in order to do RFC3261 matching. So it uses the old style
>> of comparing the headers.
>>
>
> Of course you are 100% right :)
> I didn't realize of the "wrong" magic cookie and I was just matching
> branch values.
>
> Thanks for clarify it :)
>
> PD: Then there is a bug report in the tracker that should be closed as invalid.
>
>
More information about the Devel
mailing list