diff options
| author | twilson <twilson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-03-18 15:43:34 +0000 |
|---|---|---|
| committer | twilson <twilson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-03-18 15:43:34 +0000 |
| commit | 9e8ebe6a94eb44796b296dfc214622ac583d4630 (patch) | |
| tree | a07721daf3a91733bc3e073bab0697333d426e21 /utils/astman.c | |
| parent | 421d5fe68591bbafcf04602c3bba927cac40a2a9 (diff) | |
Go through and fix a bunch of places where character strings were being interpreted as format strings. Most of these changes are solely to make compiling with -Wsecurity and -Wformat=2 happy, and were not
actual problems, per se. I also added format attributes to any printf wrapper functions I found that didn't have them. -Wsecurity and -Wmissing-format-attribute added to --enable-dev-mode.
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@109447 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'utils/astman.c')
| -rw-r--r-- | utils/astman.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/utils/astman.c b/utils/astman.c index 06fe8b3ad..2a4508589 100644 --- a/utils/astman.c +++ b/utils/astman.c @@ -141,7 +141,8 @@ static void del_chan(char *name) AST_LIST_TRAVERSE_SAFE_END; } -static void fdprintf(int fd, char *fmt, ...) + +static void __attribute__((format (printf, 2, 3))) fdprintf(int fd, char *fmt, ...) { char stuff[4096]; va_list ap; @@ -406,7 +407,8 @@ static struct message *wait_for_response(int timeout) return NULL; } -static int manager_action(char *action, char *fmt, ...) + +static int __attribute__((format (printf, 2, 3))) manager_action(char *action, char *fmt, ...) { struct ast_mansession *s; char tmp[4096]; @@ -465,7 +467,7 @@ static int hide_doing(void) static void try_status(void) { struct message *m; - manager_action("Status", ""); + manager_action("Status", "%s", ""); m = wait_for_response(10000); if (!m) { show_message("Status Failed", "Timeout waiting for response"); |