tools/xenstore: add const to the return type of canonicalize()

The return type of canonicalize() can be modified to const char *.
This avoids the need to cast the const away from the input parameter.

There need to be quite some other functions modified to take const
parameters in order to avoid further casts.

Signed-off-by: Juergen Gross <jgross@suse.com>
Reviewed-by: Julien Grall <jgrall@amazon.com>

diff --git a/tools/xenstore/xenstored_control.c b/tools/xenstore/xenstored_control.c
index 145a0e5aff43bbfd1eb9399a4384ba8e42f28545..3bdf2edc34c36160f3c86973cc6905e292628798 100644 (file)
--- a/tools/xenstore/xenstored_control.c
+++ b/tools/xenstore/xenstored_control.c
@@ -34,7 +34,7 @@
 
 struct cmd_s {
        char *cmd;
-       int (*func)(const void *, struct connection *, char **, int);
+       int (*func)(const void *, struct connection *, const char **, int);
        char *pars;
        /*
         * max_pars can be used to limit the size of the parameter vector,
@@ -47,7 +47,7 @@ struct cmd_s {
 };
 
 static int do_control_check(const void *ctx, struct connection *conn,
-                           char **vec, int num)
+                           const char **vec, int num)
 {
        if (num)
                return EINVAL;
@@ -59,7 +59,7 @@ static int do_control_check(const void *ctx, struct connection *conn,
 }
 
 static int do_control_log(const void *ctx, struct connection *conn,
-                         char **vec, int num)
+                         const char **vec, int num)
 {
        int ret;
 
@@ -126,7 +126,7 @@ static int quota_show_current(const void *ctx, struct connection *conn,
 }
 
 static int quota_set(const void *ctx, struct connection *conn,
-                    char **vec, int num, struct quota *quotas)
+                    const char **vec, int num, struct quota *quotas)
 {
        unsigned int i;
        int val;
@@ -150,7 +150,7 @@ static int quota_set(const void *ctx, struct connection *conn,
 }
 
 static int quota_get(const void *ctx, struct connection *conn,
-                    char **vec, int num)
+                    const char **vec, int num)
 {
        if (num != 1)
                return EINVAL;
@@ -159,7 +159,7 @@ static int quota_get(const void *ctx, struct connection *conn,
 }
 
 static int quota_max(const void *ctx, struct connection *conn,
-                    char **vec, int num)
+                    const char **vec, int num)
 {
        if (num > 1)
                return EINVAL;
@@ -175,7 +175,7 @@ static int quota_max(const void *ctx, struct connection *conn,
 }
 
 static int do_control_quota(const void *ctx, struct connection *conn,
-                           char **vec, int num)
+                           const char **vec, int num)
 {
        if (num == 0)
                return quota_show_current(ctx, conn, hard_quotas);
@@ -190,7 +190,7 @@ static int do_control_quota(const void *ctx, struct connection *conn,
 }
 
 static int do_control_quota_s(const void *ctx, struct connection *conn,
-                             char **vec, int num)
+                             const char **vec, int num)
 {
        if (num == 0)
                return quota_show_current(ctx, conn, soft_quotas);
@@ -203,7 +203,7 @@ static int do_control_quota_s(const void *ctx, struct connection *conn,
 
 #ifdef __MINIOS__
 static int do_control_memreport(const void *ctx, struct connection *conn,
-                               char **vec, int num)
+                               const char **vec, int num)
 {
        if (num)
                return EINVAL;
@@ -215,7 +215,7 @@ static int do_control_memreport(const void *ctx, struct connection *conn,
 }
 #else
 static int do_control_logfile(const void *ctx, struct connection *conn,
-                             char **vec, int num)
+                             const char **vec, int num)
 {
        if (num != 1)
                return EINVAL;
@@ -230,7 +230,7 @@ static int do_control_logfile(const void *ctx, struct connection *conn,
 }
 
 static int do_control_memreport(const void *ctx, struct connection *conn,
-                               char **vec, int num)
+                               const char **vec, int num)
 {
        FILE *fp;
        int fd;
@@ -270,7 +270,7 @@ static int do_control_memreport(const void *ctx, struct connection *conn,
 #endif
 
 static int do_control_print(const void *ctx, struct connection *conn,
-                           char **vec, int num)
+                           const char **vec, int num)
 {
        if (num != 1)
                return EINVAL;
@@ -281,7 +281,8 @@ static int do_control_print(const void *ctx, struct connection *conn,
        return 0;
 }
 
-static int do_control_help(const void *, struct connection *, char **, int);
+static int do_control_help(const void *, struct connection *, const char **,
+                          int);
 
 static struct cmd_s cmds[] = {
        { "check", do_control_check, "" },
@@ -322,7 +323,7 @@ static struct cmd_s cmds[] = {
 };
 
 static int do_control_help(const void *ctx, struct connection *conn,
-                          char **vec, int num)
+                          const char **vec, int num)
 {
        int cmd;
        char *resp;
@@ -348,7 +349,7 @@ int do_control(const void *ctx, struct connection *conn,
               struct buffered_data *in)
 {
        unsigned int cmd, num, off;
-       char **vec = NULL;
+       const char **vec = NULL;
 
        if (domain_is_unprivileged(conn))
                return EACCES;
@@ -365,7 +366,7 @@ int do_control(const void *ctx, struct connection *conn,
        num = xenstore_count_strings(in->buffer, in->used);
        if (cmds[cmd].max_pars)
                num = min(num, cmds[cmd].max_pars);
-       vec = talloc_array(ctx, char *, num);
+       vec = (const char **)talloc_array(ctx, char *, num);
        if (!vec)
                return ENOMEM;
        if (get_strings(in, vec, num) < num)

diff --git a/tools/xenstore/xenstored_core.c b/tools/xenstore/xenstored_core.c
index a1d3047e48959caed8af09b533818eb5e64aa774..6d27b2dd7faac4cf7b938e9939305b925225bcae 100644 (file)
--- a/tools/xenstore/xenstored_core.c
+++ b/tools/xenstore/xenstored_core.c
@@ -994,7 +994,7 @@ unsigned int get_string(const struct buffered_data *data, unsigned int offset)
  * ignore any data after the final nul.
  */
 unsigned int get_strings(struct buffered_data *data,
-                        char *vec[], unsigned int num)
+                        const char *vec[], unsigned int num)
 {
        unsigned int off, i, len;
 
@@ -1216,25 +1216,26 @@ static char *perms_to_strings(const void *ctx, const struct node_perms *perms,
        return strings;
 }
 
-char *canonicalize(struct connection *conn, const void *ctx, const char *node)
+const char *canonicalize(struct connection *conn, const void *ctx,
+                        const char *node)
 {
        const char *prefix;
 
        if (!node || (node[0] == '/') || (node[0] == '@'))
-               return (char *)node;
+               return node;
        prefix = get_implicit_path(conn);
        if (prefix)
                return talloc_asprintf(ctx, "%s/%s", prefix, node);
-       return (char *)node;
+       return node;
 }
 
 static struct node *get_node_canonicalized(struct connection *conn,
                                           const void *ctx,
                                           const char *name,
-                                          char **canonical_name,
+                                          const char **canonical_name,
                                           unsigned int perm)
 {
-       char *tmp_name;
+       const char *tmp_name;
 
        if (!canonical_name)
                canonical_name = &tmp_name;
@@ -1249,7 +1250,7 @@ static struct node *get_node_canonicalized(struct connection *conn,
 }
 
 static struct node *get_spec_node(struct connection *conn, const void *ctx,
-                                 const char *name, char **canonical_name,
+                                 const char *name, const char **canonical_name,
                                  unsigned int perm)
 {
        if (name[0] == '@')
@@ -1538,8 +1539,8 @@ static int do_write(const void *ctx, struct connection *conn,
 {
        unsigned int offset, datalen;
        struct node *node;
-       char *vec[1] = { NULL }; /* gcc4 + -W + -Werror fucks code. */
-       char *name;
+       const char *vec[1] = { NULL }; /* gcc4 + -W + -Werror fucks code. */
+       const char *name;
 
        /* Extra "strings" can be created by binary data. */
        if (get_strings(in, vec, ARRAY_SIZE(vec)) < ARRAY_SIZE(vec))
@@ -1574,7 +1575,7 @@ static int do_mkdir(const void *ctx, struct connection *conn,
                    struct buffered_data *in)
 {
        struct node *node;
-       char *name;
+       const char *name;
 
        node = get_node_canonicalized(conn, ctx, onearg(in), &name,
                                      XS_PERM_WRITE);
@@ -1703,7 +1704,7 @@ static int do_rm(const void *ctx, struct connection *conn,
 {
        struct node *node;
        int ret;
-       char *name;
+       const char *name;
        char *parentname;
 
        node = get_node_canonicalized(conn, ctx, onearg(in), &name,
@@ -1765,7 +1766,8 @@ static int do_set_perms(const void *ctx, struct connection *conn,
                        struct buffered_data *in)
 {
        struct node_perms perms, old_perms;
-       char *name, *permstr;
+       const char *name;
+       char *permstr;
        struct node *node;
 
        perms.num = xenstore_count_strings(in->buffer, in->used);

diff --git a/tools/xenstore/xenstored_core.h b/tools/xenstore/xenstored_core.h
index 84a611cbb53f484991e9cbc37c9c2c8fb8822b06..005556de90591ef7c1b4fcc06b1cfed8ed938405 100644 (file)
--- a/tools/xenstore/xenstored_core.h
+++ b/tools/xenstore/xenstored_core.h
@@ -212,7 +212,7 @@ const char *onearg(struct buffered_data *in);
 
 /* Break input into vectors, return the number, fill in up to num of them. */
 unsigned int get_strings(struct buffered_data *data,
-                        char *vec[], unsigned int num);
+                        const char *vec[], unsigned int num);
 unsigned int get_string(const struct buffered_data *data, unsigned int offset);
 
 void send_reply(struct connection *conn, enum xsd_sockmsg_type type,
@@ -224,7 +224,8 @@ void send_event(struct buffered_data *req, struct connection *conn,
 void send_ack(struct connection *conn, enum xsd_sockmsg_type type);
 
 /* Canonicalize this path if possible. */
-char *canonicalize(struct connection *conn, const void *ctx, const char *node);
+const char *canonicalize(struct connection *conn, const void *ctx,
+                        const char *node);
 
 /* Get access permissions. */
 unsigned int perm_for_conn(struct connection *conn,

diff --git a/tools/xenstore/xenstored_domain.c b/tools/xenstore/xenstored_domain.c
index 632ddb5efc180ccaa9f2d9d4edb120dd32c35395..cc67af5b98cbd7308fd5935e1b9402f1150c2ea5 100644 (file)
--- a/tools/xenstore/xenstored_domain.c
+++ b/tools/xenstore/xenstored_domain.c
@@ -1014,7 +1014,7 @@ int do_introduce(const void *ctx, struct connection *conn,
                 struct buffered_data *in)
 {
        struct domain *domain;
-       char *vec[3];
+       const char *vec[3];
        unsigned int domid;
        evtchn_port_t port;
 
@@ -1055,7 +1055,7 @@ static struct domain *find_connected_domain(unsigned int domid)
 int do_set_target(const void *ctx, struct connection *conn,
                  struct buffered_data *in)
 {
-       char *vec[2];
+       const char *vec[2];
        unsigned int domid, tdomid;
         struct domain *domain, *tdomain;
        if (get_strings(in, vec, ARRAY_SIZE(vec)) < ARRAY_SIZE(vec))

diff --git a/tools/xenstore/xenstored_lu.c b/tools/xenstore/xenstored_lu.c
index b20d0d37ea19cee9d4c9f1f52e3cfaf814cdd7c7..f7f76acbf9dc326eafe8d748fd2364f35aeb636e 100644 (file)
--- a/tools/xenstore/xenstored_lu.c
+++ b/tools/xenstore/xenstored_lu.c
@@ -360,7 +360,7 @@ static const char *lu_start(const void *ctx, struct connection *conn,
        return NULL;
 }
 
-int do_control_lu(const void *ctx, struct connection *conn, char **vec,
+int do_control_lu(const void *ctx, struct connection *conn, const char **vec,
                  int num)
 {
        const char *ret = NULL;

diff --git a/tools/xenstore/xenstored_lu.h b/tools/xenstore/xenstored_lu.h
index d633a765a31a9897daa7f86019df0b372c197e97..ac3c572ca87eb8904733445fb1ad8a3b15bb33e5 100644 (file)
--- a/tools/xenstore/xenstored_lu.h
+++ b/tools/xenstore/xenstored_lu.h
@@ -50,7 +50,7 @@ void lu_read_state(void);
 /* Write the "OK" response for the live-update command */
 unsigned int lu_write_response(FILE *fp);
 
-int do_control_lu(const void *ctx, struct connection *conn, char **vec,
+int do_control_lu(const void *ctx, struct connection *conn, const char **vec,
                  int num);
 
 /* Live update private interfaces. */
@@ -59,7 +59,7 @@ void lu_close_dump_state(struct lu_dump_state *state);
 FILE *lu_dump_open(const void *ctx);
 void lu_dump_close(FILE *fp);
 char *lu_exec(const void *ctx, int argc, char **argv);
-const char *lu_arch(const void *ctx, struct connection *conn, char **vec,
+const char *lu_arch(const void *ctx, struct connection *conn, const char **vec,
                    int num);
 const char *lu_begin(struct connection *conn);
 void lu_destroy_arch(void *data);

diff --git a/tools/xenstore/xenstored_lu_daemon.c b/tools/xenstore/xenstored_lu_daemon.c
index 310fb8c2e6b8930fcadce37718489e053e019ccb..8c7522b0e1357c5ec8d96ef4d4e7f92de7989365 100644 (file)
--- a/tools/xenstore/xenstored_lu_daemon.c
+++ b/tools/xenstore/xenstored_lu_daemon.c
@@ -118,7 +118,7 @@ static const char *lu_binary(const void *ctx, struct connection *conn,
        return NULL;
 }
 
-const char *lu_arch(const void *ctx, struct connection *conn, char **vec,
+const char *lu_arch(const void *ctx, struct connection *conn, const char **vec,
                    int num)
 {
        if (num == 2 && !strcmp(vec[0], "-f"))

diff --git a/tools/xenstore/xenstored_lu_minios.c b/tools/xenstore/xenstored_lu_minios.c
index 88fb9e0f38a24489e63e92493565ac62f8d1f56b..ae0483575e9267bf11d57470f2d3f5450a21d76c 100644 (file)
--- a/tools/xenstore/xenstored_lu_minios.c
+++ b/tools/xenstore/xenstored_lu_minios.c
@@ -104,7 +104,7 @@ static const char *lu_binary_save(const void *ctx, struct connection *conn,
        return NULL;
 }
 
-const char *lu_arch(const void *ctx, struct connection *conn, char **vec,
+const char *lu_arch(const void *ctx, struct connection *conn, const char **vec,
                    int num)
 {
        if (num == 2 && !strcmp(vec[0], "-b"))

diff --git a/tools/xenstore/xenstored_watch.c b/tools/xenstore/xenstored_watch.c
index 4195c59e17da24860b37308b252da32ba55e87d3..fefbf56ab222c6c43bd896bf4d658a0f929fbe55 100644 (file)
--- a/tools/xenstore/xenstored_watch.c
+++ b/tools/xenstore/xenstored_watch.c
@@ -161,7 +161,7 @@ static int destroy_watch(void *_watch)
 }
 
 static int check_watch_path(struct connection *conn, const void *ctx,
-                           char **path, bool *relative)
+                           const char **path, bool *relative)
 {
        /* Check if valid event. */
        if (strstarts(*path, "@")) {
@@ -184,8 +184,9 @@ static int check_watch_path(struct connection *conn, const void *ctx,
        return errno;
 }
 
-static struct watch *add_watch(struct connection *conn, char *path, char *token,
-                              bool relative, bool no_quota_check)
+static struct watch *add_watch(struct connection *conn, const char *path,
+                              const char *token, bool relative,
+                              bool no_quota_check)
 {
        struct watch *watch;
 
@@ -217,7 +218,7 @@ static struct watch *add_watch(struct connection *conn, char *path, char *token,
 int do_watch(const void *ctx, struct connection *conn, struct buffered_data *in)
 {
        struct watch *watch;
-       char *vec[2];
+       const char *vec[2];
        bool relative;
 
        if (get_strings(in, vec, ARRAY_SIZE(vec)) != ARRAY_SIZE(vec))
@@ -258,7 +259,8 @@ int do_unwatch(const void *ctx, struct connection *conn,
               struct buffered_data *in)
 {
        struct watch *watch;
-       char *node, *vec[2];
+       const char *node;
+       const char *vec[2];
 
        if (get_strings(in, vec, ARRAY_SIZE(vec)) != ARRAY_SIZE(vec))
                return EINVAL;
@@ -336,7 +338,7 @@ void read_state_watch(const void *ctx, const void *state)
 {
        const struct xs_state_watch *sw = state;
        struct connection *conn;
-       char *path, *token;
+       const char *path, *token;
        bool relative;
 
        conn = get_connection_by_id(sw->conn_id);