summaryrefslogtreecommitdiffstats
path: root/libexec
diff options
context:
space:
mode:
authoryar <yar@FreeBSD.org>2003-06-10 15:38:35 +0000
committeryar <yar@FreeBSD.org>2003-06-10 15:38:35 +0000
commit151ea28b4aa3f763472cbb41fe8b335a0ed60ecc (patch)
tree421be5bd53008e3329bd392fddb9260b8d11a58b /libexec
parent1e04cff15ef18913ddb40215e2b3abb8c133e5fe (diff)
downloadFreeBSD-src-151ea28b4aa3f763472cbb41fe8b335a0ed60ecc.zip
FreeBSD-src-151ea28b4aa3f763472cbb41fe8b335a0ed60ecc.tar.gz
Coding style fix: Use "foo = 1" instead of "foo++" to assign
the truth value to a boolean variable, especially when inside a loop. The variable can overflow otherwise, at least in theory.
Diffstat (limited to 'libexec')
-rw-r--r--libexec/getty/main.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/libexec/getty/main.c b/libexec/getty/main.c
index 5410ef7..0bb4f5e 100644
--- a/libexec/getty/main.c
+++ b/libexec/getty/main.c
@@ -356,7 +356,7 @@ main(int argc, char *argv[])
else if (islower(*p))
lower = 1;
else if (isdigit(*p))
- digit++;
+ digit = 1;
*q++ = *p++;
}
} else if (!(PL && PP))
@@ -604,7 +604,7 @@ getname(void)
np = name;
continue;
} else if (isdigit(c))
- digit++;
+ digit = 1;
if (IG && (c <= ' ' || c > 0176))
continue;
*np++ = c;
OpenPOWER on IntegriCloud