[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