Preston_PLB/freeswitch
1
0
Fork 0
forked from Mirrors/freeswitch
Code Pull Requests Activity

[mod_amqp] scan-build: Multiple dereferences of null pointer (loaded from variable 'active')

Browse Source
This commit is contained in:
Dragos Oancea 2020-02-28 13:45:28 +00:00
parent 695ba877aa
commit 746d5c3919
1 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/mod/event_handlers/mod_amqp/mod_amqp_connection.c
View File

@ -150,20 +150,26 @@ switch_status_t mod_amqp_connection_open(mod_amqp_connection_t *connections, mod
connection_attempt->password);
if (mod_amqp_log_if_amqp_error(status, "Logging in")) {
mod_amqp_connection_close(*active);
*active = NULL;
if (active) {
mod_amqp_connection_close(*active);
*active = NULL;
}
goto err;
}
// Open a channel (1). This is fairly standard
amqp_channel_open(newConnection, 1);
if (mod_amqp_log_if_amqp_error(amqp_get_rpc_reply(newConnection), "Opening channel")) {
mod_amqp_connection_close(*active);
*active = NULL;
if (active) {
mod_amqp_connection_close(*active);
*active = NULL;
}
goto err;
}
(*active)->state = newConnection;
if (active) {
(*active)->state = newConnection;
}
if (oldConnection) {
amqp_destroy_connection(oldConnection);