Fix minor misuses of snprintf
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@105841 f38db490-d61c-443f-a65b-d21fe96a405b
This commit is contained in:
parent
832983e43a
commit
701a8a40c2
|
@ -583,7 +583,7 @@ static void set_queue_variables(struct queue_ent *qe)
|
|||
if (qe->parent->callscompleted > 0)
|
||||
sl = 100 * ((float) qe->parent->callscompletedinsl / (float) qe->parent->callscompleted);
|
||||
|
||||
snprintf(interfacevar,sizeof(interfacevar),
|
||||
snprintf(interfacevar, sizeof(interfacevar),
|
||||
"QUEUENAME=%s|QUEUEMAX=%d|QUEUESTRATEGY=%s|QUEUECALLS=%d|QUEUEHOLDTIME=%d|QUEUECOMPLETED=%d|QUEUEABANDONED=%d|QUEUESRVLEVEL=%d|QUEUESRVLEVELPERF=%2.1f",
|
||||
qe->parent->name, qe->parent->maxlen, int2strat(qe->parent->strategy), qe->parent->count, qe->parent->holdtime, qe->parent->callscompleted,
|
||||
qe->parent->callsabandoned, qe->parent->servicelevel, sl);
|
||||
|
@ -2735,8 +2735,8 @@ static void update_qe_rule(struct queue_ent *qe)
|
|||
min_penalty = 0;
|
||||
if (min_penalty > max_penalty)
|
||||
min_penalty = max_penalty;
|
||||
snprintf(max_penalty_str, sizeof(max_penalty_str) - 1, "%d", max_penalty);
|
||||
snprintf(min_penalty_str, sizeof(min_penalty_str) - 1, "%d", min_penalty);
|
||||
snprintf(max_penalty_str, sizeof(max_penalty_str), "%d", max_penalty);
|
||||
snprintf(min_penalty_str, sizeof(min_penalty_str), "%d", min_penalty);
|
||||
pbx_builtin_setvar_helper(qe->chan, "QUEUE_MAX_PENALTY", max_penalty_str);
|
||||
pbx_builtin_setvar_helper(qe->chan, "QUEUE_MIN_PENALTY", min_penalty_str);
|
||||
qe->max_penalty = max_penalty;
|
||||
|
@ -3340,7 +3340,7 @@ static int try_calling(struct queue_ent *qe, const char *options, char *announce
|
|||
/* if setinterfacevar is defined, make member variables available to the channel */
|
||||
/* use pbx_builtin_setvar to set a load of variables with one call */
|
||||
if (qe->parent->setinterfacevar) {
|
||||
snprintf(interfacevar,sizeof(interfacevar), "MEMBERINTERFACE=%s|MEMBERNAME=%s|MEMBERCALLS=%d|MEMBERLASTCALL=%ld|MEMBERPENALTY=%d|MEMBERDYNAMIC=%d|MEMBERREALTIME=%d",
|
||||
snprintf(interfacevar, sizeof(interfacevar), "MEMBERINTERFACE=%s|MEMBERNAME=%s|MEMBERCALLS=%d|MEMBERLASTCALL=%ld|MEMBERPENALTY=%d|MEMBERDYNAMIC=%d|MEMBERREALTIME=%d",
|
||||
member->interface, member->membername, member->calls, (long)member->lastcall, member->penalty, member->dynamic, member->realtime);
|
||||
pbx_builtin_setvar(qe->chan, interfacevar);
|
||||
}
|
||||
|
@ -3348,7 +3348,7 @@ static int try_calling(struct queue_ent *qe, const char *options, char *announce
|
|||
/* if setqueueentryvar is defined, make queue entry (i.e. the caller) variables available to the channel */
|
||||
/* use pbx_builtin_setvar to set a load of variables with one call */
|
||||
if (qe->parent->setqueueentryvar) {
|
||||
snprintf(interfacevar,sizeof(interfacevar), "QEHOLDTIME=%ld|QEORIGINALPOS=%d",
|
||||
snprintf(interfacevar, sizeof(interfacevar), "QEHOLDTIME=%ld|QEORIGINALPOS=%d",
|
||||
(long) time(NULL) - qe->start, qe->opos);
|
||||
pbx_builtin_setvar(qe->chan, interfacevar);
|
||||
}
|
||||
|
@ -4581,7 +4581,7 @@ static int queue_function_var(struct ast_channel *chan, const char *cmd, char *d
|
|||
if (q->callscompleted > 0)
|
||||
sl = 100 * ((float) q->callscompletedinsl / (float) q->callscompleted);
|
||||
|
||||
snprintf(interfacevar,sizeof(interfacevar),
|
||||
snprintf(interfacevar, sizeof(interfacevar),
|
||||
"QUEUEMAX=%d|QUEUESTRATEGY=%s|QUEUECALLS=%d|QUEUEHOLDTIME=%d|QUEUECOMPLETED=%d|QUEUEABANDONED=%d|QUEUESRVLEVEL=%d|QUEUESRVLEVELPERF=%2.1f",
|
||||
q->maxlen, int2strat(q->strategy), q->count, q->holdtime, q->callscompleted, q->callsabandoned, q->servicelevel, sl);
|
||||
|
||||
|
|
|
@ -10607,7 +10607,7 @@ static void *pri_dchannel(void *vpri)
|
|||
}
|
||||
#endif
|
||||
|
||||
snprintf(calledtonstr, sizeof(calledtonstr)-1, "%d", e->ring.calledplan);
|
||||
snprintf(calledtonstr, sizeof(calledtonstr), "%d", e->ring.calledplan);
|
||||
pbx_builtin_setvar_helper(c, "CALLEDTON", calledtonstr);
|
||||
if (e->ring.redirectingreason >= 0)
|
||||
pbx_builtin_setvar_helper(c, "PRIREDIRECTREASON", redirectingreason2str(e->ring.redirectingreason));
|
||||
|
@ -10652,7 +10652,7 @@ static void *pri_dchannel(void *vpri)
|
|||
if (e->ring.redirectingreason >= 0)
|
||||
pbx_builtin_setvar_helper(c, "PRIREDIRECTREASON", redirectingreason2str(e->ring.redirectingreason));
|
||||
|
||||
snprintf(calledtonstr, sizeof(calledtonstr)-1, "%d", e->ring.calledplan);
|
||||
snprintf(calledtonstr, sizeof(calledtonstr), "%d", e->ring.calledplan);
|
||||
pbx_builtin_setvar_helper(c, "CALLEDTON", calledtonstr);
|
||||
|
||||
ast_mutex_lock(&pri->pvts[chanpos]->lock);
|
||||
|
@ -12522,7 +12522,7 @@ static int action_zapshowchannels(struct mansession *s, const struct message *m)
|
|||
|
||||
astman_send_ack(s, m, "Zapata channel status will follow");
|
||||
if (!ast_strlen_zero(id))
|
||||
snprintf(idText, sizeof(idText) - 1, "ActionID: %s\r\n", id);
|
||||
snprintf(idText, sizeof(idText), "ActionID: %s\r\n", id);
|
||||
|
||||
ast_mutex_lock(&iflock);
|
||||
|
||||
|
|
|
@ -306,7 +306,7 @@ static void keypad_pick_up(struct video_desc *env)
|
|||
char buf[160];
|
||||
const char *who = ast_skip_blanks(read_message(gui->bd_msg));
|
||||
buf[sizeof(buf) - 1] = '\0';
|
||||
snprintf(buf, sizeof(buf) - 1, "console dial %s", who);
|
||||
snprintf(buf, sizeof(buf), "console dial %s", who);
|
||||
ast_log(LOG_WARNING, "doing <%s>\n", buf);
|
||||
print_message(gui->bd_dialed, "\n");
|
||||
print_message(gui->bd_dialed, who);
|
||||
|
|
|
@ -2201,7 +2201,7 @@ static char *cli_complete(EditLine *el, int ch)
|
|||
len = lf->cursor - ptr;
|
||||
|
||||
if (ast_opt_remote) {
|
||||
snprintf(buf, sizeof(buf),"_COMMAND NUMMATCHES \"%s\" \"%s\"", lf->buffer, ptr);
|
||||
snprintf(buf, sizeof(buf), "_COMMAND NUMMATCHES \"%s\" \"%s\"", lf->buffer, ptr);
|
||||
fdprint(ast_consock, buf);
|
||||
res = read(ast_consock, buf, sizeof(buf));
|
||||
buf[res] = '\0';
|
||||
|
@ -2213,7 +2213,7 @@ static char *cli_complete(EditLine *el, int ch)
|
|||
/* Start with a 2048 byte buffer */
|
||||
if (!(mbuf = ast_malloc(maxmbuf)))
|
||||
return (char *)(CC_ERROR);
|
||||
snprintf(buf, sizeof(buf),"_COMMAND MATCHESARRAY \"%s\" \"%s\"", lf->buffer, ptr);
|
||||
snprintf(buf, sizeof(buf), "_COMMAND MATCHESARRAY \"%s\" \"%s\"", lf->buffer, ptr);
|
||||
fdprint(ast_consock, buf);
|
||||
res = 0;
|
||||
mbuf[0] = '\0';
|
||||
|
@ -2516,7 +2516,7 @@ static void ast_readconfig(void)
|
|||
ast_copy_string(cfg_paths.config_dir, DEFAULT_CONFIG_DIR, sizeof(cfg_paths.config_dir));
|
||||
ast_copy_string(cfg_paths.spool_dir, DEFAULT_SPOOL_DIR, sizeof(cfg_paths.spool_dir));
|
||||
ast_copy_string(cfg_paths.module_dir, DEFAULT_MODULE_DIR, sizeof(cfg_paths.module_dir));
|
||||
snprintf(cfg_paths.monitor_dir, sizeof(cfg_paths.monitor_dir) - 1, "%s/monitor", cfg_paths.spool_dir);
|
||||
snprintf(cfg_paths.monitor_dir, sizeof(cfg_paths.monitor_dir), "%s/monitor", cfg_paths.spool_dir);
|
||||
ast_copy_string(cfg_paths.var_dir, DEFAULT_VAR_DIR, sizeof(cfg_paths.var_dir));
|
||||
ast_copy_string(cfg_paths.data_dir, DEFAULT_DATA_DIR, sizeof(cfg_paths.data_dir));
|
||||
ast_copy_string(cfg_paths.log_dir, DEFAULT_LOG_DIR, sizeof(cfg_paths.log_dir));
|
||||
|
@ -2548,7 +2548,7 @@ static void ast_readconfig(void)
|
|||
ast_copy_string(cfg_paths.config_dir, v->value, sizeof(cfg_paths.config_dir));
|
||||
} else if (!strcasecmp(v->name, "astspooldir")) {
|
||||
ast_copy_string(cfg_paths.spool_dir, v->value, sizeof(cfg_paths.spool_dir));
|
||||
snprintf(cfg_paths.monitor_dir, sizeof(cfg_paths.monitor_dir) - 1, "%s/monitor", v->value);
|
||||
snprintf(cfg_paths.monitor_dir, sizeof(cfg_paths.monitor_dir), "%s/monitor", v->value);
|
||||
} else if (!strcasecmp(v->name, "astvarlibdir")) {
|
||||
ast_copy_string(cfg_paths.var_dir, v->value, sizeof(cfg_paths.var_dir));
|
||||
if (!found.dbdir)
|
||||
|
|
Reference in New Issue