AuroraMiddleware/liblzma
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Cast a char argument to isspace() to unsigned char.

Browse Source
This commit is contained in:
Lasse Collin 2009-06-24 13:01:59 +03:00
parent 1735d31ea3
commit ae82dde5d9
1 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/xz/args.c
View File

@ -374,7 +374,14 @@ parse_environment(args_info *args, char *argv0)
int argc = 1; int argc = 1;
bool prev_was_space = true; bool prev_was_space = true;
for (size_t i = 0; env[i] != '\0'; ++i) { for (size_t i = 0; env[i] != '\0'; ++i) {
if (isspace(env[i])) { // NOTE: Cast to unsigned char is needed so that correct
// value gets passed to isspace(), which expects
// unsigned char cast to int. Casting to int is done
// automatically due to integer promotion, but we need to
// force char to unsigned char manually. Otherwise 8-bit
// characters would get promoted to wrong value if
// char is signed.
if (isspace((unsigned char)env[i])) {
prev_was_space = true; prev_was_space = true;
} else if (prev_was_space) { } else if (prev_was_space) {
prev_was_space = false; prev_was_space = false;
@ -399,7 +406,7 @@ parse_environment(args_info *args, char *argv0)
argc = 1; argc = 1;
prev_was_space = true; prev_was_space = true;
for (size_t i = 0; env[i] != '\0'; ++i) { for (size_t i = 0; env[i] != '\0'; ++i) {
if (isspace(env[i])) { if (isspace((unsigned char)env[i])) {
prev_was_space = true; prev_was_space = true;
env[i] = '\0'; env[i] = '\0';
} else if (prev_was_space) { } else if (prev_was_space) {