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" <miconda(a)gmail.com>
> To: "Kamailio (SER) - Users Mailing List"
<sr-users(a)lists.kamailio.org>rg>, "Andrew Pogrebennyk"
<apogrebennyk(a)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: mcapetta(a)sipwise.com <mailto:mcapetta@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