[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[xen master] tools/console: Use const whenever we point to literal strings



commit 3beb2ee6be1d15f9f3c2bb89160803da21dc7e20
Author:     Julien Grall <jgrall@xxxxxxxxxx>
AuthorDate: Wed May 26 16:01:51 2021 +0100
Commit:     Julien Grall <jgrall@xxxxxxxxxx>
CommitDate: Wed May 26 16:01:51 2021 +0100

    tools/console: Use const whenever we point to literal strings
    
    Literal strings are not meant to be modified. So we should use const
    char * rather than char * when we want to store a pointer to them.
    
    Take the opportunity to remove the cast (char *) in console_init(). It
    is unnecessary and will remove the const.
    
    Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx>
    Acked-by: Wei Liu <wl@xxxxxxx>
    Reviewed-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
---
 tools/console/client/main.c |  4 ++--
 tools/console/daemon/io.c   | 15 ++++++++-------
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/tools/console/client/main.c b/tools/console/client/main.c
index 088be28dff..80157be421 100644
--- a/tools/console/client/main.c
+++ b/tools/console/client/main.c
@@ -325,7 +325,7 @@ int main(int argc, char **argv)
 {
        struct termios attr;
        int domid;
-       char *sopt = "hn:";
+       const char *sopt = "hn:";
        int ch;
        unsigned int num = 0;
        int opt_ind=0;
@@ -345,7 +345,7 @@ int main(int argc, char **argv)
        char *end;
        console_type type = CONSOLE_INVAL;
        bool interactive = 0;
-       char *console_names = "serial, pv, vuart";
+       const char *console_names = "serial, pv, vuart";
 
        while((ch = getopt_long(argc, argv, sopt, lopt, &opt_ind)) != -1) {
                switch(ch) {
diff --git a/tools/console/daemon/io.c b/tools/console/daemon/io.c
index 4af27ffc5d..200b575d76 100644
--- a/tools/console/daemon/io.c
+++ b/tools/console/daemon/io.c
@@ -87,14 +87,14 @@ struct buffer {
 };
 
 struct console {
-       char *ttyname;
+       const char *ttyname;
        int master_fd;
        int master_pollfd_idx;
        int slave_fd;
        int log_fd;
        struct buffer buffer;
        char *xspath;
-       char *log_suffix;
+       const char *log_suffix;
        int ring_ref;
        xenevtchn_handle *xce_handle;
        int xce_pollfd_idx;
@@ -109,9 +109,9 @@ struct console {
 };
 
 struct console_type {
-       char *xsname;
-       char *ttyname;
-       char *log_suffix;
+       const char *xsname;
+       const char *ttyname;
+       const char *log_suffix;
        bool optional;
        bool use_gnttab;
 };
@@ -813,7 +813,8 @@ static int console_init(struct console *con, struct domain 
*dom, void **data)
        int err = -1;
        struct timespec ts;
        struct console_type **con_type = (struct console_type **)data;
-       char *xsname, *xspath;
+       const char *xsname;
+       char *xspath;
 
        if (clock_gettime(CLOCK_MONOTONIC, &ts) < 0) {
                dolog(LOG_ERR, "Cannot get time of day %s:%s:L%d",
@@ -835,7 +836,7 @@ static int console_init(struct console *con, struct domain 
*dom, void **data)
        con->log_suffix = (*con_type)->log_suffix;
        con->optional = (*con_type)->optional;
        con->use_gnttab = (*con_type)->use_gnttab;
-       xsname = (char *)(*con_type)->xsname;
+       xsname = (*con_type)->xsname;
        xspath = xs_get_domain_path(xs, dom->domid);
        s = realloc(xspath, strlen(xspath) +
                    strlen(xsname) + 1);
--
generated by git-patchbot for /home/xen/git/xen.git#master



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.