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