From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Thu, 12 Jul 2007 23:51:10 +0000 (+0000)
Subject: Fix a portability bug (ye olde not casting a <ctype.h> argument to
X-Git-Tag: REL8_3_BETA1~459
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=04b54876b65fe9c138c2fcc67fa2d028b0afae94;p=postgresql

Fix a portability bug (ye olde not casting a <ctype.h> argument to
unsigned char).  Fortunately we still have buildfarm machines that
will flag this.  Seems to be new in CVS HEAD, so no backpatch.
---

diff --git a/src/backend/utils/adt/cash.c b/src/backend/utils/adt/cash.c
index 06dc4fec8c..ed37332346 100644
--- a/src/backend/utils/adt/cash.c
+++ b/src/backend/utils/adt/cash.c
@@ -13,7 +13,7 @@
  * this version handles 64 bit numbers and so can hold values up to
  * $92,233,720,368,547,758.07.
  *
- * $PostgreSQL: pgsql/src/backend/utils/adt/cash.c,v 1.70 2007/02/27 23:48:07 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/utils/adt/cash.c,v 1.71 2007/07/12 23:51:10 tgl Exp $
  */
 
 #include "postgres.h"
@@ -240,8 +240,9 @@ cash_in(PG_FUNCTION_ARGS)
 		}
 	}
 
-	/* should only be trailing digits followed by whitespace or closing paren */
-	while (isdigit(*s)) s++;
+	/* should only be trailing digits followed by whitespace or right paren */
+	while (isdigit((unsigned char) *s))
+		s++;
 	while (isspace((unsigned char) *s) || *s == ')')
 		s++;