Thanks for the report and for the fix proposal.
However it is not clear to me how the crash is happening. In case of timeout 
(like it looks in your case) [this 
line](https://github.com/kamailio/kamailio/blob/5801ff9612d9eef9a7b06f91ff533a59020181e9/src/modules/http_async_client/http_multi.c#L110)
 set sockptr to NULL so that, when sock_cb is called, "cell" iS ULL 
and [thsi 
condition](https://github.com/kamailio/kamailio/blob/5801ff9612d9eef9a7b06f91ff533a59020181e9/src/modules/http_async_client/http_multi.c#L167)
 is false. From the backtrace we see that this is not the case in your crash 
because sockptr still holds the pointer to the cell that has already be 
unlinked and freed. I'm not sure what's happening in your case but:
- which version of curl are you using?
- could you please clarify/describe the scenario?

On the patch side: you are right that we could, in sock_cb, lookup for the cell 
by the easy handler, there would be no need of a cell_tmp to compare with in 
this case.
Anyway this scenario should never occur this way so I would like to understand 
more.


-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4020#issuecomment-2466262446
You are receiving this because you are subscribed to this thread.

Message ID: <kamailio/kamailio/pull/4020/c2466262...@github.com>
_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org

Reply via email to