libctf: fix isspace casts
authorNick Alcock <nick.alcock@oracle.com>
Mon, 13 Jul 2020 15:05:15 +0000 (16:05 +0100)
committerNick Alcock <nick.alcock@oracle.com>
Wed, 22 Jul 2020 17:05:32 +0000 (18:05 +0100)
isspace() notoriously takes an int, not a char.  Cast uses
appropriately.

libctf/
* ctf-lookup.c (ctf_lookup_by_name): Adjust.

libctf/ChangeLog
libctf/ctf-lookup.c

index cfb9d01b80322176304d42bf14bda99f396e27b4..9183d2b7d6538e5d3f62999c12a93f418fb8e73d 100644 (file)
@@ -1,3 +1,7 @@
+2020-07-22  Nick Alcock  <nick.alcock@oracle.com>
+
+       * ctf-lookup.c (ctf_lookup_by_name): Adjust.
+
 2020-07-22  Nick Alcock  <nick.alcock@oracle.com>
 
        * ctf-archive.c (ctf_arc_bufopen): Endian-swap the archive magic
index 8daab632dca3ae64f6c754e8455925be2fe3f0b4..57fb6c580ee297d1567650181a84d70eca5f2bbd 100644 (file)
@@ -83,7 +83,7 @@ ctf_lookup_by_name (ctf_file_t *fp, const char *name)
 
   for (p = name, end = name + strlen (name); *p != '\0'; p = q)
     {
-      while (isspace (*p))
+      while (isspace ((int) *p))
        p++;                    /* Skip leading whitespace.  */
 
       if (p == end)
@@ -133,13 +133,13 @@ ctf_lookup_by_name (ctf_file_t *fp, const char *name)
               strncmp (p, lp->ctl_prefix, (size_t) (q - p)) == 0) &&
              (size_t) (q - p) >= lp->ctl_len)
            {
-             for (p += lp->ctl_len; isspace (*p); p++)
+             for (p += lp->ctl_len; isspace ((int) *p); p++)
                continue;       /* Skip prefix and next whitespace.  */
 
              if ((q = strchr (p, '*')) == NULL)
                q = end;        /* Compare until end.  */
 
-             while (isspace (q[-1]))
+             while (isspace ((int) q[-1]))
                q--;            /* Exclude trailing whitespace.  */
 
              /* Expand and/or allocate storage for a slice of the name, then