Make one-bit fields unsigned instead of signed. This has no effect,

since they are only tested for zero/nonzero; but it's arguably a bad
idea to set a {-1, 0} variable to 1 (as happens in this code).

Found by:	Coverity Prevent
This commit is contained in:
Colin Percival 2008-06-09 14:41:28 +00:00
parent af58f6feff
commit f6dd73207f

View file

@ -42,7 +42,7 @@ struct subst_rule {
struct subst_rule *next;
regex_t re;
char *result;
int global:1, print:1, symlink:1;
unsigned int global:1, print:1, symlink:1;
};
struct substitution {