Hello,
the patch should be fine for now. My review is mainly targeting to see
if other tm callbacks suffer from same situation and if I can find a
better solution. But for your specific case, should be ok to use as it
is now.
Cheers,
Daniel
On 20.11.17 12:13, Marco Capetta wrote:
> Hi Daniel,
>
> did you have time to review the fix, or can I use the last suggested
> patch?
>
> Thanks!
>
> Cheers,
> Marco
>
> On 11/14/2017 08:39 AM, Daniel-Constantin Mierla wrote:
>> Hello,
>>
>> ok, thanks for testing, good to know the issue was identified! I will do
>> another review soon to see if there would be a better fix for the case.
>>
>> Cheers,
>> Daniel
>>
>>
>> On 13.11.17 09:46, Marco Capetta wrote:
>>> Hi Daniel,
>>>
>>> we left the test system running the whole weekend and so far we haven't see
>>> any new ACC record for provisional response messages.
>>> I think that the latest patch solved the issue.
>>>
>>> Thank you
>>>
>>> Cheers
>>> Marco
>>>
>>> ----- Original Message -----
>>> From: "Daniel-Constantin Mierla" <mico...@gmail.com>
>>> To: "Kamailio (SER) - Users Mailing List" <sr-users@lists.kamailio.org>,
>>> "Andrew Pogrebennyk" <apogreben...@sipwise.com>
>>> Sent: Friday, November 10, 2017 9:48:29 AM
>>> Subject: Re: [SR-Users] 183 acc records even if early_media equals to 0
>>>
>>>
>>>
>>> On 10.11.17 09:42, Andrew Pogrebennyk wrote:
>>>> On 11/10/2017 09:34 AM, Marco Capetta wrote:
>>>>> +
>>>>> if(relayed_msg->REPLY_STATUS==uas_rb->activ_type) {
>>>>> +
>>>>> run_trans_callbacks_with_buf( TMCB_RESPONSE_OUT, uas_rb, t->uas.request,
>>>>> +
>>>>>
>>>>> relayed_msg, relayed_code);
>>>>> + }
>>>> my guess is that in case of fake reply like a 408 the relayed_msg is -1
>>>> so addressing the member by pointer fails.
>>>>
>>>> I have suggested Marco to try with if(relayed_msg==FAKED_REPLY ||
>>>> relayed_msg->REPLY_STATUS==uas_rb->activ_type)
>>>>
>>>> Not sure if there are any other cases where relayed_msg is a NULL but we
>>>> will test this and let you know if it helps.
>>> Indeed, I noticed that is the FAKED_REPLY looking at the locals in the
>>> backtrace he sent to me. I think it is better to use relayed_code
>>> instead of relayed_msg->REPLY_STATUS in that condition.
>>>
>>> Cheers,
>>> Daniel
>>>
>
> --
> *Marco Capetta *
> Operations Engineer
>
> Sipwise GmbH <http://www.sipwise.com> , Campus 21/Europaring F15
> AT-2345 Brunn am Gebirge
>
> Phone: +43(0)1 301 2044 <tel:+4313012044>
> Email: mcape...@sipwise.com <mailto:mcape...@sipwise.com>
> Website: www.sipwise.com <http://www.sipwise.com>
>
> Particulars according Austrian Companies Code paragraph 14
> "Sipwise GmbH" - Europaring F15 - 2345 Brunn am Gebirge
> FN:305595f, Commercial Court Vienna, ATU64002206
>
--
Daniel-Constantin Mierla
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio Advanced Training - www.asipto.com
Kamailio World Conference - May 14-16, 2018 - www.kamailioworld.com
_______________________________________________
Kamailio (SER) - Users Mailing List
sr-users@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
