diff options
author | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-03-07 06:36:33 +0000 |
---|---|---|
committer | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-03-07 06:36:33 +0000 |
commit | 2d66183adf20c8b3898a3bfc97627d55dff7891f (patch) | |
tree | e1d83f46856427c7e9df46336f418ec059a92c57 /main/manager.c | |
parent | 6623be8860eb9337ebbc3a5a32388270fac5628b (diff) |
Safely use the strncat() function.
(closes issue #11958)
Reported by: norman
Patches:
20080209__bug11958.diff.txt uploaded by Corydon76 (license 14)
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@106552 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'main/manager.c')
-rw-r--r-- | main/manager.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/main/manager.c b/main/manager.c index b78ecf5e3..4fadb744c 100644 --- a/main/manager.c +++ b/main/manager.c @@ -206,10 +206,10 @@ static char *authority_to_str(int authority, char *res, int reslen) for (i = 0; i < (sizeof(perms) / sizeof(perms[0])) - 1; i++) { if (authority & perms[i].num) { if (*res) { - strncat(res, ",", (reslen > running_total) ? reslen - running_total : 0); + strncat(res, ",", (reslen > running_total) ? reslen - running_total - 1 : 0); running_total++; } - strncat(res, perms[i].label, (reslen > running_total) ? reslen - running_total : 0); + strncat(res, perms[i].label, (reslen > running_total) ? reslen - running_total - 1 : 0); running_total += strlen(perms[i].label); } } |