summaryrefslogtreecommitdiffstats
path: root/usr.bin
diff options
context:
space:
mode:
authorcperciva <cperciva@FreeBSD.org>2008-06-09 14:41:28 +0000
committercperciva <cperciva@FreeBSD.org>2008-06-09 14:41:28 +0000
commit95bad3329f6acab7fea233b66e858274828ed69d (patch)
tree5f1d2fa6f56266d25bac41c9cda69365d191d414 /usr.bin
parent02aa7dcfd94c8f1fe8486cf86260e55edf5a0d63 (diff)
downloadFreeBSD-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.c2
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 {
OpenPOWER on IntegriCloud