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 we to store a pointer to them.
Signed-off-by: Julien Grall <jgr...@amazon.com> --- tools/misc/xen-detect.c | 2 +- tools/misc/xenhypfs.c | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/misc/xen-detect.c b/tools/misc/xen-detect.c index eac9e46a35bb..18b28dabf311 100644 --- a/tools/misc/xen-detect.c +++ b/tools/misc/xen-detect.c @@ -44,7 +44,7 @@ enum guest_type { XEN_NONE = 3 }; -static char *type; +static const char *type; static char *ver; static void cpuid(uint32_t idx, uint32_t *regs, int pv_context) diff --git a/tools/misc/xenhypfs.c b/tools/misc/xenhypfs.c index 5da24aed905c..df398b07bdc0 100644 --- a/tools/misc/xenhypfs.c +++ b/tools/misc/xenhypfs.c @@ -81,9 +81,9 @@ static int xenhypfs_wr(char *path, char *val) return ret; } -static char *xenhypfs_type(struct xenhypfs_dirent *ent) +static const char *xenhypfs_type(struct xenhypfs_dirent *ent) { - char *res; + const char *res; switch (ent->type) { case xenhypfs_type_dir: @@ -134,7 +134,7 @@ static int xenhypfs_ls(char *path) return ret; } -static int xenhypfs_tree_sub(char *path, unsigned int depth) +static int xenhypfs_tree_sub(const char *path, unsigned int depth) { struct xenhypfs_dirent *ent; unsigned int n, i; -- 2.17.1