kenmirrors
/
freeswitch
mirror of https://github.com/signalwire/freeswitch.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

CID:1214196 Dereference null return value, if we hit this condition, something very bad has happened, we should never have the private not set if we get to the hangup hook, log error instead of crashing

This commit is contained in:
Michael Jerris 2014-05-15 14:09:38 +00:00
parent d2884ac5a6
commit 4e888a282c
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/mod/applications/mod_spy/mod_spy.c
View File

@ -59,6 +59,11 @@ static switch_status_t spy_on_hangup(switch_core_session_t *session)
const char *uuid = switch_core_session_get_uuid(session);
spy_t *spy = NULL, *p = NULL, *prev = NULL;
if (!data) {
switch_log_printf(SWITCH_CHANNEL_SESSION_LOG(session), SWITCH_LOG_ERROR, "Can't call mod_spy hangup hook due to missing private data\n");
return SWITCH_STATUS_SUCCESS;
}
switch_thread_rwlock_wrlock(globals.spy_hash_lock);
spy = switch_core_hash_find(globals.spy_hash, data);