From: Julien Grall <jgr...@amazon.com>

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.

Signed-off-by: Julien Grall <jgr...@amazon.com>
---
 tools/console/client/main.c |  4 ++--
 tools/console/daemon/io.c   | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/tools/console/client/main.c b/tools/console/client/main.c
index 088be28dff02..80157be42144 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 4af27ffc5d02..6a8a94e31b65 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;
 };
-- 
2.17.1


Reply via email to