aboutsummaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorAlex Richardson <arichardson@FreeBSD.org>2021-01-19 11:35:04 +0000
committerAlex Richardson <arichardson@FreeBSD.org>2021-01-19 21:23:25 +0000
commit0348c8fcfaa22d4f2bf548ad9c187c09ef90c533 (patch)
tree743b8bf3b822a32712f27910687562491dca6a42 /lib
parent74eb7f97ecd62d0a7e056e4c2fd696f2c267e3a5 (diff)
downloadsrc-0348c8fcfaa22d4f2bf548ad9c187c09ef90c533.tar.gz
src-0348c8fcfaa22d4f2bf548ad9c187c09ef90c533.zip
getopt: Fix conversion from string-literal to non-const char *
Define a non-const static char EMSG[] = "" to avoid having to add __DECONST() to all uses of EMSG. Also make current_dash a const char * to fix this warning.
Diffstat (limited to 'lib')
-rw-r--r--lib/libc/stdlib/getopt.c3
-rw-r--r--lib/libc/stdlib/getopt_long.c4
2 files changed, 3 insertions, 4 deletions
diff --git a/lib/libc/stdlib/getopt.c b/lib/libc/stdlib/getopt.c
index b6bdf8a1eeaf..b7f2229a2f54 100644
--- a/lib/libc/stdlib/getopt.c
+++ b/lib/libc/stdlib/getopt.c
@@ -54,8 +54,7 @@ char *optarg; /* argument associated with option */
#define BADCH (int)'?'
#define BADARG (int)':'
-#define EMSG ""
-
+static char EMSG[] = "";
/*
* getopt --
* Parse argc/argv argument vector.
diff --git a/lib/libc/stdlib/getopt_long.c b/lib/libc/stdlib/getopt_long.c
index 4d92fd0cd45d..1f3548bef9ad 100644
--- a/lib/libc/stdlib/getopt_long.c
+++ b/lib/libc/stdlib/getopt_long.c
@@ -88,7 +88,7 @@ char *optarg; /* argument associated with option */
#define BADARG ((*options == ':') ? (int)':' : (int)'?')
#define INORDER (int)1
-#define EMSG ""
+static char EMSG[] = "";
#ifdef GNU_COMPATIBLE
#define NO_PREFIX (-1)
@@ -194,7 +194,7 @@ parse_long_options(char * const *nargv, const char *options,
{
char *current_argv, *has_equal;
#ifdef GNU_COMPATIBLE
- char *current_dash;
+ const char *current_dash;
#endif
size_t current_argv_len;
int i, match, exact_match, second_partial_match;