From 0b5c7307a7b6eb8ba2dc88bf446bb47eeb3d0f7c Mon Sep 17 00:00:00 2001 From: Dragos Oancea Date: Fri, 6 Mar 2020 13:40:03 +0000 Subject: [PATCH] [mod_portaudio] scan-build: Access to field 'next' results in a dereference of a null pointer (loaded from variable 'previous'). Value stored to 'codec_ms' is never read. --- src/mod/endpoints/mod_portaudio/mod_portaudio.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/mod/endpoints/mod_portaudio/mod_portaudio.c b/src/mod/endpoints/mod_portaudio/mod_portaudio.c index d3d941257e..0e24faef16 100644 --- a/src/mod/endpoints/mod_portaudio/mod_portaudio.c +++ b/src/mod/endpoints/mod_portaudio/mod_portaudio.c @@ -644,7 +644,9 @@ static void remove_stream(audio_stream_t * stream, int already_locked) for (previous = globals.stream_list; previous && previous->next && previous->next != stream; previous = previous->next) { ; } - previous->next = stream->next; + if (previous) { + previous->next = stream->next; + } } if (! already_locked) { switch_mutex_unlock(globals.streams_lock); @@ -1180,7 +1182,6 @@ static switch_call_cause_t channel_outgoing_channel(switch_core_session_t *sessi } if (outbound_profile->destination_number && !strncasecmp(outbound_profile->destination_number, "endpoint", sizeof("endpoint")-1)) { - codec_ms = -1; endpoint = NULL; endpoint_name = switch_core_strdup(outbound_profile->pool, outbound_profile->destination_number); endpoint_name = strchr(endpoint_name, '/');