diff options
| author | Terry Wilson <twilson@digium.com> | 2008-03-18 15:43:34 +0000 |
|---|---|---|
| committer | Terry Wilson <twilson@digium.com> | 2008-03-18 15:43:34 +0000 |
| commit | b02bc230af68d05df171a0a099d6c3d94168ab64 (patch) | |
| tree | a07721daf3a91733bc3e073bab0697333d426e21 /utils/astman.c | |
| parent | 760fc3403c7355fa05bc1b7470405239aad7af8e (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: https://origsvn.digium.com/svn/asterisk/trunk@109447 65c4cc65-6c06-0410-ace0-fbb531ad65f3
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"); |