summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBrent Cook <bcook@cvs.openbsd.org>2015-01-19 20:47:04 +0000
committerBrent Cook <bcook@cvs.openbsd.org>2015-01-19 20:47:04 +0000
commit36c9b1bb0ce06669996b92f34f0cbdafe094776e (patch)
tree09b801395bf34d674276743440e83cfd868e4a49
parentb0ad00aeede0f706ad0525479e24873628a25d80 (diff)
Use initial assignment of action to check for errors.
This simplifies things and make action = -1 no longer a dead store. Also, spell FALLTHROUGH consistently. reported by fritjof@alokat.org
-rw-r--r--usr.sbin/ntpd/client.c4
-rw-r--r--usr.sbin/ntpd/ntpd.c9
2 files changed, 6 insertions, 7 deletions
diff --git a/usr.sbin/ntpd/client.c b/usr.sbin/ntpd/client.c
index 0d984a3def0..06c41fda259 100644
--- a/usr.sbin/ntpd/client.c
+++ b/usr.sbin/ntpd/client.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: client.c,v 1.97 2015/01/09 23:44:07 bcook Exp $ */
+/* $OpenBSD: client.c,v 1.98 2015/01/19 20:47:03 bcook Exp $ */
/*
* Copyright (c) 2003, 2004 Henning Brauer <henning@openbsd.org>
@@ -85,7 +85,7 @@ client_addr_init(struct ntp_peer *p)
break;
default:
fatalx("king bula sez: wrong AF in client_addr_init");
- /* not reached */
+ /* NOTREACHED */
}
}
diff --git a/usr.sbin/ntpd/ntpd.c b/usr.sbin/ntpd/ntpd.c
index 9796c34133e..6f85b14266d 100644
--- a/usr.sbin/ntpd/ntpd.c
+++ b/usr.sbin/ntpd/ntpd.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: ntpd.c,v 1.86 2015/01/14 21:14:27 naddy Exp $ */
+/* $OpenBSD: ntpd.c,v 1.87 2015/01/19 20:47:03 bcook Exp $ */
/*
* Copyright (c) 2003, 2004 Henning Brauer <henning@openbsd.org>
@@ -578,12 +578,11 @@ ctl_main(int argc, char *argv[])
case 'a':
action = CTL_SHOW_ALL;
break;
- default:
- usage();
- /* NOTREACHED */
}
- } else
+ }
+ if (action == -1)
usage();
+ /* NOTREACHED */
if ((fd = socket(AF_UNIX, SOCK_STREAM, 0)) == -1)
err(1, "ntpctl: socket");