kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-09-30 10:33:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

Coverity Report: Fix issues for error type REVERSE_INULL (deprecated modules)

Browse Source 
* Fix only issue pointed out by deprecated_REVERSE_INULL.txt for
app_meetme.c in find_user().

* Change use of %i to %d in sscanf() in find_user().  The use of %i gives
unexpected parsing because it can accept hex, octal, and decimal integer
formats.

* Changed other uses of %i in app_meetme() to use %d for consistency.

(issue ASTERISK-19648)
Reported by: Matt Jordan
........

Merged revisions 367906 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 367907 from http://svn.asterisk.org/svn/asterisk/branches/10


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@367908 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Richard Mudgett
2012-05-29 22:37:19 +00:00
parent 94187aafc0
commit 77f5e86e4d
1 changed files with 32 additions and 33 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
apps/app_meetme.c
View File

@@ -3147,7 +3147,7 @@ static int conf_run(struct ast_channel *chan, struct ast_conference *conf, struc
"Channel: %s\r\n" "Channel: %s\r\n"
"Uniqueid: %s\r\n" "Uniqueid: %s\r\n"
"Meetme: %s\r\n" "Meetme: %s\r\n"
"Usernum: %i\r\n" "Usernum: %d\r\n"
"Status: on\r\n", "Status: on\r\n",
ast_channel_name(chan), ast_channel_uniqueid(chan), conf->confno, user->user_no); ast_channel_name(chan), ast_channel_uniqueid(chan), conf->confno, user->user_no);
} }
@@ -3165,7 +3165,7 @@ static int conf_run(struct ast_channel *chan, struct ast_conference *conf, struc
"Channel: %s\r\n" "Channel: %s\r\n"
"Uniqueid: %s\r\n" "Uniqueid: %s\r\n"
"Meetme: %s\r\n" "Meetme: %s\r\n"
"Usernum: %i\r\n" "Usernum: %d\r\n"
"Status: off\r\n", "Status: off\r\n",
ast_channel_name(chan), ast_channel_uniqueid(chan), conf->confno, user->user_no); ast_channel_name(chan), ast_channel_uniqueid(chan), conf->confno, user->user_no);
} }
@@ -3178,7 +3178,7 @@ static int conf_run(struct ast_channel *chan, struct ast_conference *conf, struc
"Channel: %s\r\n" "Channel: %s\r\n"
"Uniqueid: %s\r\n" "Uniqueid: %s\r\n"
"Meetme: %s\r\n" "Meetme: %s\r\n"
"Usernum: %i\r\n" "Usernum: %d\r\n"
"Status: on\r\n", "Status: on\r\n",
ast_channel_name(chan), ast_channel_uniqueid(chan), conf->confno, user->user_no); ast_channel_name(chan), ast_channel_uniqueid(chan), conf->confno, user->user_no);
} }
@@ -3191,7 +3191,7 @@ static int conf_run(struct ast_channel *chan, struct ast_conference *conf, struc
"Channel: %s\r\n" "Channel: %s\r\n"
"Uniqueid: %s\r\n" "Uniqueid: %s\r\n"
"Meetme: %s\r\n" "Meetme: %s\r\n"
"Usernum: %i\r\n" "Usernum: %d\r\n"
"Status: off\r\n", "Status: off\r\n",
ast_channel_name(chan), ast_channel_uniqueid(chan), conf->confno, user->user_no); ast_channel_name(chan), ast_channel_uniqueid(chan), conf->confno, user->user_no);
} }
@@ -4579,8 +4579,7 @@ static struct ast_conf_user *find_user(struct ast_conference *conf, const char *
struct ast_conf_user *user = NULL; struct ast_conf_user *user = NULL;
int cid; int cid;
sscanf(callerident, "%30i", &cid); if (conf && callerident && sscanf(callerident, "%30d", &cid) == 1) {
if (conf && callerident) {
user = ao2_find(conf->usercontainer, &cid, 0); user = ao2_find(conf->usercontainer, &cid, 0);
/* reference decremented later in admin_exec */ /* reference decremented later in admin_exec */
return user; return user;