diff options
author | cperciva <cperciva@FreeBSD.org> | 2008-06-09 14:41:28 +0000 |
---|---|---|
committer | cperciva <cperciva@FreeBSD.org> | 2008-06-09 14:41:28 +0000 |
commit | 95bad3329f6acab7fea233b66e858274828ed69d (patch) | |
tree | 5f1d2fa6f56266d25bac41c9cda69365d191d414 /usr.bin | |
parent | 02aa7dcfd94c8f1fe8486cf86260e55edf5a0d63 (diff) | |
download | FreeBSD-src-95bad3329f6acab7fea233b66e858274828ed69d.zip FreeBSD-src-95bad3329f6acab7fea233b66e858274828ed69d.tar.gz |
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
Diffstat (limited to 'usr.bin')
-rw-r--r-- | usr.bin/tar/subst.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/usr.bin/tar/subst.c b/usr.bin/tar/subst.c index ea96d15..1c24d06 100644 --- a/usr.bin/tar/subst.c +++ b/usr.bin/tar/subst.c @@ -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 { |