diff options
| author | Richard Mudgett <rmudgett@digium.com> | 2012-05-31 18:39:30 +0000 |
|---|---|---|
| committer | Richard Mudgett <rmudgett@digium.com> | 2012-05-31 18:39:30 +0000 |
| commit | dd2427c1414d62cd7e3c867f17de9d6dbac844fe (patch) | |
| tree | 5760fb673e236d0f58b094240de81ca9780cd135 /main/tcptls.c | |
| parent | fdb002a43a819542693815767a0ca9aba135f18c (diff) | |
Coverity Report: Fix issues for error type REVERSE_INULL (core modules)
* Fixes findings: 0-2,5,7-15,24-26,28-31
(issue ASTERISK-19648)
Reported by: Matt Jordan
........
Merged revisions 368039 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 368042 from http://svn.asterisk.org/svn/asterisk/branches/10
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@368052 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'main/tcptls.c')
| -rw-r--r-- | main/tcptls.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/main/tcptls.c b/main/tcptls.c index a6e444d8b..4988da96a 100644 --- a/main/tcptls.c +++ b/main/tcptls.c @@ -255,7 +255,7 @@ static void *handle_tcptls_connection(void *data) return NULL; } - if (tcptls_session && tcptls_session->parent->worker_fn) { + if (tcptls_session->parent->worker_fn) { return tcptls_session->parent->worker_fn(tcptls_session); } else { return tcptls_session; @@ -457,9 +457,7 @@ client_start_error: close(desc->accept_fd); desc->accept_fd = -1; } - if (tcptls_session) { - ao2_ref(tcptls_session, -1); - } + ao2_ref(tcptls_session, -1); return NULL; } |