diff options
author | yar <yar@FreeBSD.org> | 2003-06-10 15:38:35 +0000 |
---|---|---|
committer | yar <yar@FreeBSD.org> | 2003-06-10 15:38:35 +0000 |
commit | 151ea28b4aa3f763472cbb41fe8b335a0ed60ecc (patch) | |
tree | 421be5bd53008e3329bd392fddb9260b8d11a58b /libexec/getty | |
parent | 1e04cff15ef18913ddb40215e2b3abb8c133e5fe (diff) | |
download | FreeBSD-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/getty')
-rw-r--r-- | libexec/getty/main.c | 4 |
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; |