gaaf commented on this pull request.


> @@ -269,6 +271,25 @@ int populate_leg_info( struct dlg_cell *dlg, struct 
> sip_msg *msg,
 
        if (rr_set.s) pkg_free(rr_set.s);

print_rr_body() allocates the memory for rrset.s, it is up to the caller to 
free it. As it may be allocated twice, it must be freed twice and that must be 
done before the next call to print_rr_body(). Each free is in the same scope as 
the call to print_rr_body(). I fail to see how these can be combined into one 
free.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/1832#discussion_r303425969
_______________________________________________
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to