From 31c10fac222f070f288dfa91cd4a7517396fa5f0 Mon Sep 17 00:00:00 2001 From: Andrey Volk Date: Thu, 1 Apr 2021 16:25:10 +0300 Subject: [PATCH] [mod_kazoo] scan-build: Argument with 'nonnull' attribute passed null in kz_endpoint_outgoing_channel() --- src/mod/event_handlers/mod_kazoo/kazoo_endpoints.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/mod/event_handlers/mod_kazoo/kazoo_endpoints.c b/src/mod/event_handlers/mod_kazoo/kazoo_endpoints.c index cb2b28cb26..4ddfeddae4 100644 --- a/src/mod/event_handlers/mod_kazoo/kazoo_endpoints.c +++ b/src/mod/event_handlers/mod_kazoo/kazoo_endpoints.c @@ -222,20 +222,20 @@ static switch_call_cause_t kz_endpoint_outgoing_channel(switch_core_session_t *s } if (!call_fwd_is_valid) { - dest = strdup(endpoint_dial); + dest = endpoint_dial ? strdup(endpoint_dial) : strdup(""); } else if (call_fwd_is_failover) { switch_log_printf(SWITCH_CHANNEL_SESSION_LOG(session), SWITCH_LOG_DEBUG, "setting failover => %s\n", callforward_dial); - dest = strdup(endpoint_dial); + dest = endpoint_dial ? strdup(endpoint_dial) : strdup(""); failover_dial = callforward_dial; } else if (call_fwd_is_substitute) { - switch_log_printf(SWITCH_CHANNEL_SESSION_LOG(session), SWITCH_LOG_DEBUG, "setting call fwd substitute => %s\n", callforward_dial); - dest = strdup(callforward_dial); + dest = callforward_dial ? strdup(callforward_dial) : strdup(""); + switch_log_printf(SWITCH_CHANNEL_SESSION_LOG(session), SWITCH_LOG_DEBUG, "setting call fwd substitute => %s\n", dest); } else { dest = switch_mprintf("%s%s%s", endpoint_dial ? endpoint_dial : "", (endpoint_dial ? endpoint_separator ? endpoint_separator : "," : ""), callforward_dial); switch_log_printf(SWITCH_CHANNEL_SESSION_LOG(session), SWITCH_LOG_DEBUG, "setting call fwd append => %s => %s\n", callforward_dial, dest); } } else { - dest = strdup(endpoint_dial); + dest = endpoint_dial ? strdup(endpoint_dial) : strdup(""); } dialed_user = (char *)switch_xml_attr(x_user, "id"); -- 2.47.3