src/sulogin.c: Invert logic to reduce indentation
Also, it was checking for >=0 for success, but since that code is for opening a different tty as stdin, that was bogus. But since it's guaranteed to be either 0 or -1, this commit doesn't add any code to make sure it's 0 (i.e., we could say !=0 instead of ==-1). That's more appropriate for a different commit. Signed-off-by: Alejandro Colomar <alx@kernel.org>
This commit is contained in:
committed by
Serge Hallyn
parent
efd169e010
commit
88760598f0
+4
-6
@@ -84,12 +84,10 @@ main(int argc, char *argv[])
|
||||
close(1);
|
||||
close(2);
|
||||
|
||||
if (open(argv[1], O_RDWR) >= 0) {
|
||||
dup (0);
|
||||
dup (0);
|
||||
} else {
|
||||
exit (1);
|
||||
}
|
||||
if (open(argv[1], O_RDWR) == -1)
|
||||
exit(1);
|
||||
dup(0);
|
||||
dup(0);
|
||||
}
|
||||
if (access (PASSWD_FILE, F_OK) == -1) { /* must be a password file! */
|
||||
(void) puts (_("No password file"));
|
||||
|
||||
Reference in New Issue
Block a user