Fix ctype arguments.

Cast arguments to the various ctype functions to unsigned char to match their
documentation.

Reviewed-by: Andy Polyakov <appro@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/4203)

diff --git a/apps/opt.c b/apps/opt.c
index 72d0c535fa924e06afa8ad90c027c7f9bdeb5bd3..40d6a279b3dfa36b0b7f50c9f956f8671adc054d 100644 (file)
--- a/apps/opt.c
+++ b/apps/opt.c
@@ -61,7 +61,7 @@ char *opt_progname(const char *argv0)
     if (n > sizeof prog - 1)
         n = sizeof prog - 1;
     for (q = prog, i = 0; i < n; i++, p++)
-        *q++ = isupper(*p) ? tolower(*p) : *p;
+        *q++ = tolower((unsigned char)*p);
     *q = '\0';
     return prog;
 }

diff --git a/apps/rehash.c b/apps/rehash.c
index ad7108aad9511ab1c08d439ade5f8323d8c4f7d1..8bb9a14ef6e65f65769a8884894609353dc90e38 100644 (file)
--- a/apps/rehash.c
+++ b/apps/rehash.c
@@ -300,7 +300,7 @@ static int massage_filename(char *name)
 
     if (q != NULL) {
         for (q++; *q != '\0'; q++) {
-            if (!isdigit(*q))
+            if (!isdigit((unsigned char)*q))
                 return 1;
         }
     }

diff --git a/ssl/ssl_ciph.c b/ssl/ssl_ciph.c
index 914d0d8f4aa6e1dab25626a4096a962fe96a51df..60e1308d540135b4992fdb1d354702bac063972b 100644 (file)
--- a/ssl/ssl_ciph.c
+++ b/ssl/ssl_ciph.c
@@ -1001,7 +1001,8 @@ static int ssl_cipher_process_rulestr(const char *rule_str,
                    ((ch >= 'a') && (ch <= 'z')) ||
                    (ch == '-') || (ch == '.') || (ch == '='))
 #else
-            while (isalnum(ch) || (ch == '-') || (ch == '.') || (ch == '='))
+            while (isalnum((unsigned char)ch) || (ch == '-') || (ch == '.')
+                   || (ch == '='))
 #endif
             {
                 ch = *(++l);

diff --git a/test/ecstresstest.c b/test/ecstresstest.c
index 7c76f02108bfac22adbce45b5ffad4a430ff15fc..b47095cba01c38a9a208d0f488ef6580df639abd 100644 (file)
--- a/test/ecstresstest.c
+++ b/test/ecstresstest.c
@@ -109,7 +109,7 @@ static int atoi64(const char *in, int64_t *result)
     for ( ; *in != '\0'; in++) {
         char c = *in;
 
-        if (!isdigit(c))
+        if (!isdigit((unsigned char)c))
             return 0;
         ret *= 10;
         ret += (c - '0');

diff --git a/test/evp_test.c b/test/evp_test.c
index 3875081c39096039d789fd860b7db8d892ca973c..9de7fcca6e00c22c327ee1578ca4f8f305379cf7 100644 (file)
--- a/test/evp_test.c
+++ b/test/evp_test.c
@@ -1302,7 +1302,7 @@ static int parse_uint64(const char *value, uint64_t *pr)
             return -1;
         }
         *pr *= 10;
-        if (!TEST_true(isdigit(*p))) {
+        if (!TEST_true(isdigit((unsigned char)*p))) {
             TEST_error("Invalid character in string %s", value);
             return -1;
         }

diff --git a/test/testutil/format_output.c b/test/testutil/format_output.c
index 13123d5c6a6a767d8c11b8d49208a1eef261cd79..9b04a94f4205e696be4febf759ff873594f8238a 100644 (file)
--- a/test/testutil/format_output.c
+++ b/test/testutil/format_output.c
@@ -72,12 +72,12 @@ static void test_fail_string_common(const char *prefix, const char *file,
         if (l1 > 0) {
             b1[n1 = l1 > width ? width : l1] = 0;
             for (i = 0; i < n1; i++)
-                b1[i] = isprint(m1[i]) ? m1[i] : '.';
+                b1[i] = isprint((unsigned char)m1[i]) ? m1[i] : '.';
         }
         if (l2 > 0) {
             b2[n2 = l2 > width ? width : l2] = 0;
             for (i = 0; i < n2; i++)
-                b2[i] = isprint(m2[i]) ? m2[i] : '.';
+                b2[i] = isprint((unsigned char)m2[i]) ? m2[i] : '.';
         }
         diff = 0;
         i = 0;

diff --git a/test/testutil/stanza.c b/test/testutil/stanza.c
index 35967528f8312c4c918175a6d3f6d2a558b55b4d..09fc18108079194f0a70d347095b081012bada95 100644 (file)
--- a/test/testutil/stanza.c
+++ b/test/testutil/stanza.c
@@ -70,12 +70,12 @@ static char *strip_spaces(char *p)
     char *q;
 
     /* Skip over leading spaces */
-    while (*p && isspace(*p))
+    while (*p && isspace((unsigned char)*p))
         p++;
     if (!*p)
         return NULL;
 
-    for (q = p + strlen(p) - 1; q != p && isspace(*q); )
+    for (q = p + strlen(p) - 1; q != p && isspace((unsigned char)*q); )
         *q-- = '\0';
     return *p ? p : NULL;
 }