Closed #1407.
--
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/1407#event-2743896810___
Kamailio (SER) - Development Mailing List
I am closing it, long time no activity and no similar reports. Reopen if still
needs to be addressed.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
@grumvalski , @giavac , @dragos-oancea - can this be closed or there is still
something here to work on?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
I've been able to reproduce the crash that lead to Dragos analysis and this PR,
both in debian 8 and in CentOS 7.
But I tracked it down to a different cause, which is `libcurl` crashing when
using the threaded resolver for DNS lookups. This happens even with latest
`libcurl`.
The work around
@giavac - any updates on this? Were you able to reproduce this with a newer
5.1.x version?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Pinging again to see if anything else was left to be addressed here.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
FYI this was kamailio `5.1.0` on debian 8, with stock libcurl (`7.38`).
I'm trying to reproduce the issue with `5.1.2` and both libcurl `7.38` and
`7.59` (still on debian 8) (including with restarts under load), and I will
update here with any relevant information.
--
You are receiving this
@dragos-oancea, @grumvalski - did this one ended in some conclusion? Merging or
not?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
is it possible is being set to NULL before headers.t being freed in some case ?
I noticed there are checks inside set_query_params() for other already
allocated memory buffers . It's crashing randomly at startup, in this
scenario, I dont have backtraces anymore.
--
You are receiving this
set_quey_params is called in mod_init by init_query_params
(https://github.com/kamailio/kamailio/blob/master/src/modules/http_async_client/async_http.c#L631),
so whenever ah_set_req is called the query_params struct should already have
been initialized.
headers.t is a double pointer, the
@grumvalski , set_query_params() is called in 3 places, for us kamailio is
crashing at startup, and I suspect header_list_add(_params.headers,
>rs); is sometimes called before set_query_params(_params) in
ah_set_req() , thus leading to a kamailio crash at startup, under traffic.
--
You are
11 matches
Mail list logo
