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/manager.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/manager.c')
-rw-r--r-- | main/manager.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/main/manager.c b/main/manager.c index 7a1c9a83b..2861c6d9b 100644 --- a/main/manager.c +++ b/main/manager.c @@ -6360,7 +6360,7 @@ static int auth_http_callback(struct ast_tcptls_session_instance *ser, ast_md5_hash(resp_hash, resp); } - if (!d.nonce || strncasecmp(d.response, resp_hash, strlen(resp_hash))) { + if (strncasecmp(d.response, resp_hash, strlen(resp_hash))) { /* Something was wrong, so give the client to try with a new challenge */ AST_RWLIST_UNLOCK(&users); nonce = 0; |