[sr-dev] Remote-Party-ID Parsed Header Crash

[Daniel-Constantin Mierla]

On 16/01/2017 00:48, Joshua Colp wrote:
> On Sat, Jan 14, 2017, at 06:39 PM, Joshua Colp wrote:
>> On Sat, Jan 14, 2017, at 06:20 PM, Daniel-Constantin Mierla wrote:
>>> If you want a solution to avoid the issue via config: in the
>>> request_route, store the RPID in an avp and update the acc parameter to
>>> use the avp instead of the rpid variable.
>>>
>>> As for code, a solution could be resetting the 'hdr->parsed' pointers to
>>> NULL that are in private memory after memcpy(&tmsg, req,
>>> sizeof(sip_msg_t)); -- using same kind of loop like at the end of
>>> acc_onreply(), but wihout clean_hdr_field(). In this way it ensure that
>>> the acc is using only what it parses and it is freeing only those
>>> headers.
> We've been testing the configuration change and as expected it has
> resolved the problem. Thanks Daniel!
>
Welcome! I will try to fix it properly in the code as well.

Cheers,
Daniel

-- 
Daniel-Constantin Mierla
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference - May 8-10, 2017 - www.kamailioworld.com