Bug #330
closed
isprint() returns false for "legacy" code, results in bad prompt
0%
Description
So after my mondo locale push, I noticed that sftp, and several other programs, were putting \076 at the end of the prompt instead of '>'. Basically, all libtecla consumers are affected.
Upon analysis, the problem is that isprint() returns false for a variety of characters in non-C locales.
The problem stems from the fact that certain legacy code uses a macro version of isprint(), that relies on these characters having ispunct set.
The UTF-8 locale data doesn't set that for these characters because, after all, they are not punctuation characters but other kinds of symbols. (Although notably the POSIX standard insists that ispunct() return true for them in the POSIX locale.)
Note that programs built with -D_XPG4 or that use the functional versions (by #undef ispunct) get the right value.
The simple solution is to force these characters to be punctuation in the same way that we force certain other characters to be digits or other type data.
At some point, it would be good if libtecla were compiled with the XPG4 versions of the macros, as well, so that it got more accurate type data. I'll file that in a separate bug.
Files
Updated by Garrett D'Amore over 10 years ago
Here's the fix, and a test program is attached.
diff -r feb49cca530d usr/src/cmd/localedef/ctype.c
--- a/usr/src/cmd/localedef/ctype.c Sun Oct 10 08:20:45 2010 -0700
+++ b/usr/src/cmd/localedef/ctype.c Sun Oct 10 16:53:28 2010 -0700
@@ -244,6 +244,20 @@
ctn->ctype |= _ISXDIGIT;
if (strchr(" \t", (char)wc))
ctn->ctype |= _ISBLANK;
+
+ /*
+ * Technically these settings are only
+ * required for the C locale. However, it
+ * turns out that because of the historical
+ * version of isprint(), we need them for all
+ * locales as well. Note that these are not
+ * necessarily valid punctation characters in
+ * the current language, but ispunct() needs
+ * to return TRUE for them.
+ */
+ if (strchr("!\"'#$%&()*+,-./:;<=>?@[\\]^_`{|}~",
+ (char)wc))
+ ctn->ctype |= _ISPUNCT;
}
/*
Updated by Garrett D'Amore over 10 years ago
- Category set to lib - userland libraries
Updated by
Updated by Garrett D'Amore over 10 years ago
- Status changed from Closed to Resolved
This needs to stay resolved, not closed.