diff options
| author | Noah Misch <noah@leadboat.com> | 2014-07-06 00:29:51 -0400 |
|---|---|---|
| committer | Noah Misch <noah@leadboat.com> | 2014-07-06 00:30:11 -0400 |
| commit | 49c279efe7858454114bd304444e95573be0804b (patch) | |
| tree | 32d94898357a5c5001b8e4ac079241a0774ffda0 | |
| parent | e44964c709b4a5f18d3f9a8fe6d5c259091315c1 (diff) | |
Consistently pass an "unsigned char" to ctype.h functions.
The isxdigit() calls relied on undefined behavior. The isascii() call
was well-defined, but our prevailing style is to include the cast.
Back-patch to 9.4, where the isxdigit() calls were introduced.
| -rw-r--r-- | contrib/pg_upgrade/controldata.c | 2 | ||||
| -rw-r--r-- | src/backend/utils/adt/json.c | 7 |
2 files changed, 6 insertions, 3 deletions
diff --git a/contrib/pg_upgrade/controldata.c b/contrib/pg_upgrade/controldata.c index 2906ccbf8c8..13c95a2e2e9 100644 --- a/contrib/pg_upgrade/controldata.c +++ b/contrib/pg_upgrade/controldata.c @@ -154,7 +154,7 @@ get_control_data(ClusterInfo *cluster, bool live_check) if (GET_MAJOR_VERSION(cluster->major_version) <= 803) { for (p = bufin; *p; p++) - if (!isascii(*p)) + if (!isascii((unsigned char) *p)) pg_fatal("The 8.3 cluster's pg_controldata is incapable of outputting ASCII, even\n" "with LANG=C. You must upgrade this cluster to a newer version of PostgreSQL\n" "8.3 to fix this bug. PostgreSQL 8.3.7 and later are known to work properly.\n"); diff --git a/src/backend/utils/adt/json.c b/src/backend/utils/adt/json.c index 972a22f65e5..a64e3c7a613 100644 --- a/src/backend/utils/adt/json.c +++ b/src/backend/utils/adt/json.c @@ -2353,8 +2353,11 @@ escape_json(StringInfo buf, const char *str) * only unicode escape that should be present is \u0000, * all the other unicode escapes will have been resolved. */ - if (p[1] == 'u' && isxdigit(p[2]) && isxdigit(p[3]) - && isxdigit(p[4]) && isxdigit(p[5])) + if (p[1] == 'u' && + isxdigit((unsigned char) p[2]) && + isxdigit((unsigned char) p[3]) && + isxdigit((unsigned char) p[4]) && + isxdigit((unsigned char) p[5])) appendStringInfoCharMacro(buf, *p); else appendStringInfoString(buf, "\\\\"); |