From: Erez Zadok <ezk@cs.sunysb.edu>
Date: Wed, 14 Sep 2005 14:14:37 +0000 (+0000)
Subject: cast fixes for isFOO
X-Git-Tag: before-clocktime-fixes~1
X-Git-Url: https://git.fsl.cs.stonybrook.edu/?a=commitdiff_plain;h=f66b6e10e1cd5560c73992034deefd4e099684c6;p=am-utils-6.0.git

cast fixes for isFOO
---

diff --git a/ChangeLog b/ChangeLog
index 51277dd..c4f8977 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -4,8 +4,8 @@
 	(strsplit), libamu/hasmntopt.c (nextmntopt), amd/sun_map.c
 	(sun_entry2amd), amd/sun2amd.c (sun2amd_convert_convert),
 	amd/mapc.c (mapc_add_kv), amd/info_file.c (file_search_or_reload),
-	amd/info_exec.c (exec_parse_qanswer): cast isspace() arg to
-	unsigned char, which is safer as per Christos.
+	amd/info_exec.c (exec_parse_qanswer): cast isspace() and isascii()
+	arg to unsigned char, which is safer as per Christos.
 
 	* amd/sun_map.c (sun_entry2amd): isspace takes an int, not a char.
 
diff --git a/amd/info_file.c b/amd/info_file.c
index 0acd09b..1af2415 100644
--- a/amd/info_file.c
+++ b/amd/info_file.c
@@ -79,7 +79,7 @@ file_read_line(char *buf, int size, FILE *fp)
 	 * Skip leading white space on next line
 	 */
 	while ((ch = getc(fp)) != EOF &&
-	       isascii(ch) && isspace(ch)) ;
+	       isascii((unsigned char)ch) && isspace((unsigned char)ch)) ;
 	(void) ungetc(ch, fp);
       } else {
 	return done;
@@ -133,7 +133,7 @@ file_search_or_reload(mnt_map *m,
     /*
      * Find start of key
      */
-    for (kp = key_val; *kp && isascii(*kp) && isspace((unsigned char)*kp); kp++) ;
+    for (kp = key_val; *kp && isascii((unsigned char)*kp) && isspace((unsigned char)*kp); kp++) ;
 
     /*
      * Ignore blank lines
@@ -144,7 +144,7 @@ file_search_or_reload(mnt_map *m,
     /*
      * Find end of key
      */
-    for (cp = kp; *cp && (!isascii(*cp) || !isspace((unsigned char)*cp)); cp++) ;
+    for (cp = kp; *cp && (!isascii((unsigned char)*cp) || !isspace((unsigned char)*cp)); cp++) ;
 
     /*
      * Check whether key matches
@@ -153,7 +153,7 @@ file_search_or_reload(mnt_map *m,
       *cp++ = '\0';
 
     if (fn || (*key == *kp && STREQ(key, kp))) {
-      while (*cp && isascii(*cp) && isspace((unsigned char)*cp))
+      while (*cp && isascii((unsigned char)*cp) && isspace((unsigned char)*cp))
 	cp++;
       if (*cp) {
 	/*
diff --git a/libamu/strutil.c b/libamu/strutil.c
index 5e93ee3..8f2c4ee 100644
--- a/libamu/strutil.c
+++ b/libamu/strutil.c
@@ -139,7 +139,7 @@ strsplit(char *s, int ch, int qc)
     /*
      * skip to split char
      */
-    while (*s && (ch == ' ' ? (isascii(*s) && isspace((unsigned char)*s)) : *s == ch))
+    while (*s && (ch == ' ' ? (isascii((unsigned char)*s) && isspace((unsigned char)*s)) : *s == ch))
       *s++ = '\0';
 
     /*
@@ -156,7 +156,7 @@ strsplit(char *s, int ch, int qc)
     /*
      * skip to split char
      */
-    while (*s && !(ch == ' ' ? (isascii(*s) && isspace((unsigned char)*s)) : *s == ch)) {
+    while (*s && !(ch == ' ' ? (isascii((unsigned char)*s) && isspace((unsigned char)*s)) : *s == ch)) {
       if (*s++ == qc) {
 	/*
 	 * Skip past string.
diff --git a/mk-amd-map/mk-amd-map.c b/mk-amd-map/mk-amd-map.c
index 9119cbe..740f8e0 100644
--- a/mk-amd-map/mk-amd-map.c
+++ b/mk-amd-map/mk-amd-map.c
@@ -93,7 +93,7 @@ read_line(char *buf, int size, FILE *fp)
 	/*
 	 * Skip leading white space on next line
 	 */
-	while ((ch = getc(fp)) != EOF && isascii(ch) && isspace(ch)) ;
+	while ((ch = getc(fp)) != EOF && isascii((unsigned char)ch) && isspace((unsigned char)ch)) ;
 	(void) ungetc(ch, fp);
       } else {
 	return done;
@@ -144,7 +144,7 @@ read_file(FILE *fp, char *map, voidp db)
     /*
      * Find start of key
      */
-    for (kp = key_val; *kp && isascii(*kp) && isspace((unsigned char)*kp); kp++) ;
+    for (kp = key_val; *kp && isascii((unsigned char)*kp) && isspace((unsigned char)*kp); kp++) ;
 
     /*
      * Ignore blank lines
@@ -155,7 +155,7 @@ read_file(FILE *fp, char *map, voidp db)
     /*
      * Find end of key
      */
-    for (cp = kp; *cp && (!isascii(*cp) || !isspace((unsigned char)*cp)); cp++) ;
+    for (cp = kp; *cp && (!isascii((unsigned char)*cp) || !isspace((unsigned char)*cp)); cp++) ;
 
     /*
      * Check whether key matches, or whether
@@ -163,7 +163,7 @@ read_file(FILE *fp, char *map, voidp db)
      */
     if (*cp)
       *cp++ = '\0';
-    while (*cp && isascii(*cp) && isspace((unsigned char)*cp))
+    while (*cp && isascii((unsigned char)*cp) && isspace((unsigned char)*cp))
       cp++;
     if (*kp == '+') {
       fprintf(stderr, "Can't interpolate %s\n", kp);