diff options
author | hasso <hasso> | 2004-10-07 21:40:25 +0000 |
---|---|---|
committer | hasso <hasso> | 2004-10-07 21:40:25 +0000 |
commit | dda095222f30c8bee6edc46f2c1babb66f9e699d (patch) | |
tree | 25b33dc3ed6fc992f5cf8b8e055da1ddc8d2b747 | |
parent | ea8e9d972e8775f3e1c4e03fbacb18a42e8c4177 (diff) |
Vtysh compiles cleanly as well.
-rw-r--r-- | vtysh/ChangeLog | 7 | ||||
-rw-r--r-- | vtysh/vtysh.c | 17 | ||||
-rw-r--r-- | vtysh/vtysh.h | 4 | ||||
-rw-r--r-- | vtysh/vtysh_config.c | 12 | ||||
-rw-r--r-- | vtysh/vtysh_main.c | 1 |
5 files changed, 22 insertions, 19 deletions
diff --git a/vtysh/ChangeLog b/vtysh/ChangeLog index 5d8b7e44..80c2295b 100644 --- a/vtysh/ChangeLog +++ b/vtysh/ChangeLog @@ -1,3 +1,10 @@ +2004-10-07 Hasso Tepper <hasso at quagga.net> + + * vtysh.c, vtysh.h, vtysh_config.c, vtysh_main.c: Fix compiler + warnings: make strings const, signed -> unsigned, remove unused + variables. + * vtysh_config.c: Fix crash introduced with previous patch. + 2004-10-03 Hasso Tepper <hasso at quagga.net> * vtsyh_main.c: Enter into enable node by default. Disable node doesn't diff --git a/vtysh/vtysh.c b/vtysh/vtysh.c index 79e07420..733bb437 100644 --- a/vtysh/vtysh.c +++ b/vtysh/vtysh.c @@ -161,7 +161,7 @@ vtysh_client_config (struct vtysh_client *vclient, char *line) } int -vtysh_client_execute (struct vtysh_client *vclient, char *line, FILE *fp) +vtysh_client_execute (struct vtysh_client *vclient, const char *line, FILE *fp) { int ret; char buf[1001]; @@ -245,7 +245,7 @@ vtysh_pager_init () /* Command execution over the vty interface. */ void -vtysh_execute_func (char *line, int pager) +vtysh_execute_func (const char *line, int pager) { int ret, cmd_stat; vector vline; @@ -390,13 +390,13 @@ vtysh_execute_func (char *line, int pager) } void -vtysh_execute_no_pager (char *line) +vtysh_execute_no_pager (const char *line) { vtysh_execute_func (line, 0); } void -vtysh_execute (char *line) +vtysh_execute (const char *line) { vtysh_execute_func (line, 1); } @@ -512,7 +512,7 @@ int vtysh_rl_describe () { int ret; - int i; + unsigned int i; vector vline; vector describe; int width; @@ -1576,9 +1576,6 @@ DEFUN (vtysh_write_memory, { int ret = CMD_SUCCESS; char line[] = "write memory\n"; - char *vtysh_conf; - extern struct host host; - FILE *fp; /* If integrated Quagga.conf explicitely set. */ if (vtysh_writeconfig_integrated) @@ -1700,7 +1697,7 @@ DEFUN (vtysh_show_running_daemons, /* Execute command in child process. */ int -execute_command (char *command, int argc, char *arg1, char *arg2) +execute_command (const char *command, int argc, char *arg1, char *arg2) { int ret; pid_t pid; @@ -1871,7 +1868,7 @@ vtysh_install_default (enum node_type node) /* Making connection to protocol daemon. */ int -vtysh_connect (struct vtysh_client *vclient, char *path) +vtysh_connect (struct vtysh_client *vclient, const char *path) { int ret; int sock, len; diff --git a/vtysh/vtysh.h b/vtysh/vtysh.h index b632b6de..e0c3a5b7 100644 --- a/vtysh/vtysh.h +++ b/vtysh/vtysh.h @@ -51,8 +51,8 @@ void vtysh_connect_all (); void vtysh_readline_init (); void vtysh_user_init (); -void vtysh_execute (char *); -void vtysh_execute_no_pager (char *); +void vtysh_execute (const char *); +void vtysh_execute_no_pager (const char *); char *vtysh_prompt (); diff --git a/vtysh/vtysh_config.c b/vtysh/vtysh_config.c index 26879186..f068b1fd 100644 --- a/vtysh/vtysh_config.c +++ b/vtysh/vtysh_config.c @@ -83,7 +83,7 @@ config_del (struct config* config) } struct config * -config_get (int index, char *line) +config_get (int index, const char *line) { struct config *config; struct config *config_loop; @@ -122,13 +122,13 @@ config_get (int index, char *line) } void -config_add_line (struct list *config, char *line) +config_add_line (struct list *config, const char *line) { listnode_add (config, XSTRDUP (MTYPE_VTYSH_CONFIG_LINE, line)); } void -config_add_line_uniq (struct list *config, char *line) +config_add_line_uniq (struct list *config, const char *line) { struct listnode *nn; char *pnt; @@ -142,7 +142,7 @@ config_add_line_uniq (struct list *config, char *line) } void -vtysh_config_parse_line (char *line) +vtysh_config_parse_line (const char *line) { char c; static struct config *config = NULL; @@ -294,7 +294,7 @@ vtysh_config_dump (FILE *fp) struct config *config; struct list *master; char *line; - int i; + unsigned int i; LIST_LOOP (config_top, line, nn) { @@ -403,7 +403,7 @@ vtysh_read_config (char *config_default_dir) void vtysh_config_write () { - char *line; + char line[81]; extern struct host host; if (host.name) diff --git a/vtysh/vtysh_main.c b/vtysh/vtysh_main.c index 5f7856cf..7f8e0592 100644 --- a/vtysh/vtysh_main.c +++ b/vtysh/vtysh_main.c @@ -190,7 +190,6 @@ main (int argc, char **argv, char **env) int eval_flag = 0; int boot_flag = 0; char *eval_line = NULL; - char *integrated_file = NULL; /* Preserve name of myself. */ progname = ((p = strrchr (argv[0], '/')) ? ++p : argv[0]); |