From c62193c5defdcf496cb66740d8a3cfd81e501435 Mon Sep 17 00:00:00 2001 From: Kevin Harwell Date: Mon, 19 Oct 2020 15:31:14 -0500 Subject: [PATCH] res_pjsip, res_pjsip_session: initialize local variables This patch initializes a couple of local variables to some default values. Interestingly, in the 'pj_status_t dlg_status' case the value not being initialized caused memory to grow, and not be recovered, in the off nominal path (at least on my machine). Change-Id: I22ee65e1e1bff8efacea8a167c6c8428898523f7 --- res/res_pjsip/pjsip_transport_management.c | 2 +- res/res_pjsip_session.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/res/res_pjsip/pjsip_transport_management.c b/res/res_pjsip/pjsip_transport_management.c index af572d3b6f..e14c8e93d6 100644 --- a/res/res_pjsip/pjsip_transport_management.c +++ b/res/res_pjsip/pjsip_transport_management.c @@ -78,7 +78,7 @@ static void keepalive_transport_send_keepalive(struct monitored_transport *monit static void *keepalive_transport_thread(void *data) { struct ao2_container *transports; - pj_thread_desc desc; + pj_thread_desc desc = { 0 }; pj_thread_t *thread; if (pj_thread_register("Asterisk Keepalive Thread", desc, &thread) != PJ_SUCCESS) { diff --git a/res/res_pjsip_session.c b/res/res_pjsip_session.c index 82234aa538..ec0114566d 100644 --- a/res/res_pjsip_session.c +++ b/res/res_pjsip_session.c @@ -3747,7 +3747,7 @@ static pjsip_inv_session *pre_session_setup(pjsip_rx_data *rdata, const struct a pjsip_dialog *dlg; pjsip_inv_session *inv_session; unsigned int options = endpoint->extensions.flags; - pj_status_t dlg_status; + pj_status_t dlg_status = PJ_EUNKNOWN; if (pjsip_inv_verify_request(rdata, &options, NULL, NULL, ast_sip_get_pjsip_endpoint(), &tdata) != PJ_SUCCESS) { if (tdata) {