@vingarzan commented on this pull request.
> tmb.t_request_within(&uac_r);
}
- if(h.s)
- pkg_free(h.s);
- if(content.s) {
- pkg_free(content.s);
+ watcher_contact = 0;
Shouldn't - both branches of the `if` before have the watcher_contact saved on
the transaction. I think that freeing would result in a double-free, so instead
we leave it for the `uac_request_cb` to clean up after the response or timeout
is reached.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3895#discussion_r1660941267
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3895/review/[email protected]>_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to [email protected]
