Add __attribute__((__noreturn__)) to various function prototypes in usr.sbin/.
authorSascha Wildner <saw@online.de>
Sat, 13 Jan 2018 20:37:56 +0000 (21:37 +0100)
committerSascha Wildner <saw@online.de>
Sat, 13 Jan 2018 20:40:20 +0000 (21:40 +0100)
This general cleanup is useful in various ways. It helps the compiler
optimize better and it also helps later GCCs detect better when to give
-Wimplicit-fallthrough warnings and when not.

Found-with: -Wsuggest-attribute=noreturn

39 files changed:
usr.sbin/ac/ac.c
usr.sbin/arp/arp.c
usr.sbin/autofs/common.h
usr.sbin/btconfig/btconfig.c
usr.sbin/burncd/burncd.c
usr.sbin/cdcontrol/cdcontrol.c
usr.sbin/clog/clog.c
usr.sbin/dntpd/client.h
usr.sbin/fdcontrol/fdcontrol.c
usr.sbin/ftp-proxy/ftp-proxy.c
usr.sbin/gifconfig/gifconfig.c
usr.sbin/inetd/builtins.c
usr.sbin/lpr/common_source/lp.h
usr.sbin/lpr/lpc/extern.h
usr.sbin/lpr/lpd/extern.h
usr.sbin/mld6query/mld6query.c
usr.sbin/mrouted/mtrace.c
usr.sbin/ndiscvt/inf-token.l
usr.sbin/ndp/ndp.c
usr.sbin/pfctl/pfctl.c
usr.sbin/pflogd/pflogd.c
usr.sbin/ppp/main.h
usr.sbin/pwd_mkdb/pwd_mkdb.c
usr.sbin/route6d/route6d.c
usr.sbin/rpc.lockd/lockd.c
usr.sbin/rpc.ypupdated/ypupdated_main.c
usr.sbin/rpc.ypxfrd/ypxfrd_main.c
usr.sbin/rpcbind/rpcbind.h
usr.sbin/rwhod/rwhod.c
usr.sbin/sensorsd/sensorsd.c
usr.sbin/setkey/setkey.c
usr.sbin/setkey/token.l
usr.sbin/sysvipcd/sysvipcd.c
usr.sbin/timed/timed/extern.h
usr.sbin/timed/timedc/extern.h
usr.sbin/traceroute/traceroute.c
usr.sbin/traceroute6/traceroute6.c
usr.sbin/watchdogd/watchdogd.c
usr.sbin/ypbind/ypbind.c

index c403a84..b61bff2 100644 (file)
@@ -94,7 +94,7 @@ static void           show_today(struct user_list *, struct utmp_list *,
                            time_t);
 static void            show_users(struct user_list *);
 static struct user_list        *update_user(struct user_list *, char *, time_t);
-static void            usage(void);
+static void            usage(void) __dead2;
 
 /*
  * open wtmp or die
index bc20745..025239a 100644 (file)
@@ -75,7 +75,7 @@ static void print_entry(struct sockaddr_dl *sdl,
 static void nuke_entry(struct sockaddr_dl *sdl,
        struct sockaddr_inarp *addr, struct rt_msghdr *rtm);
 static int delete(char *host, char *info);
-static void usage(void);
+static void usage(void) __dead2;
 static int set(int argc, char **argv);
 static int get(char *host);
 static int file(char *name);
index 063d9b6..5cc4ec3 100644 (file)
@@ -113,8 +113,8 @@ void        defined_parse_and_add(char *def);
 void   lesser_daemon(void);
 
 int    main_automount(int argc, char **argv);
-void   main_automountd(int argc, char **argv);
-void   main_autounmountd(int argc, char **argv);
+void   main_automountd(int argc, char **argv) __dead2;
+void   main_autounmountd(int argc, char **argv) __dead2;
 
 FILE   *auto_popen(const char *argv0, ...);
 int    auto_pclose(FILE *iop);
index adf4ce7..b1c2fb8 100644 (file)
@@ -59,9 +59,9 @@ struct result {
        int8_t          rssi;
 };
 
-static void badarg(const char *);
-static void badparam(const char *);
-static void usage(void);
+static void badarg(const char *) __dead2;
+static void badparam(const char *) __dead2;
+static void usage(void) __dead2;
 static int set_unit(unsigned long);
 static void config_unit(void);
 static void print_info(int);
index 8ae4d40..1d882c4 100644 (file)
@@ -63,7 +63,7 @@ static int write_file(struct track_info *);
 static int roundup_blocks(struct track_info *);
 static void cue_ent(struct cdr_cue_entry *, int, int, int, int, int, int, int);
 static void cleanup(int);
-static void usage(void);
+static void usage(void) __dead2;
 
 int
 main(int argc, char **argv)
index a419a44..c077869 100644 (file)
@@ -136,7 +136,7 @@ static int           play_blocks(int blk, int len);
 static int              run(int cmd, char *arg);
 static char            *parse(char *buf, int *cmd);
 static void             help(void);
-static void             usage(void);
+static void             usage(void) __dead2;
 static char            *use_cdrom_instead(const char *);
 static const char      *strstatus(int);
 static u_int            dbprog_discid(void);
index 50791da..7f5df60 100644 (file)
@@ -56,9 +56,9 @@
  */
 #define BUFFER_SIZE 16384
 
-static void init_log(const char *lname, size_t size);
-static void read_log(const char *lname, int optf);
-static void usage(void);
+static void init_log(const char *lname, size_t size) __dead2;
+static void read_log(const char *lname, int optf) __dead2;
+static void usage(void) __dead2;
 
 static const char *pname;
 
index 1120d96..5c9ae3f 100644 (file)
@@ -123,7 +123,7 @@ struct server_info {
 typedef struct server_info *server_info_t;
 
 void client_init(void);
-void client_main(struct server_info **info_ary, int count);
+void client_main(struct server_info **info_ary, int count) __dead2;
 void client_poll(server_info_t info, int poll_interval, 
                  int calc_offset_correction);
 void client_check(struct server_info **check, 
index 7c0bb79..fccce5a 100644 (file)
@@ -47,7 +47,7 @@ getnumber(void)
   return i;
 }
 
-static void
+__dead2 static void
 usage(void)
 {
   fprintf(stderr, "usage: fdcontrol [-d 0|1] | [-s] device-node\n");
index f07d8e7..6c90ece 100644 (file)
@@ -98,10 +98,10 @@ void        client_read(struct session *);
 void   client_write(struct session *);
 int    drop_privs(void);
 void   end_session(struct session *);
-void   exit_daemon(void);
+void   exit_daemon(void) __dead2;
 int    get_line(char *, size_t *);
 void   handle_connection(const int);
-void   handle_signal(int);
+void   handle_signal(int) __dead2;
 struct session * init_session(void);
 void   logmsg(int, const char *, ...) __printflike(2, 3);
 u_int16_t parse_port(int);
@@ -112,7 +112,7 @@ void        server_read(struct session *);
 void   server_write(struct session *);
 int    allow_data_connection(struct session *s);
 const char *sock_ntop(struct sockaddr *);
-void   usage(void);
+void   usage(void) __dead2;
 
 char linebuf[MAX_LINE + 1];
 size_t linelen;
index 9c5b18c..5744917 100644 (file)
@@ -127,7 +127,7 @@ static void in_status(int);
 static void in6_status(int);
 #endif
 static void ether_status(int);
-static void Perror(char *);
+static void Perror(char *) __dead2;
 static void in_getaddr(char *, int);
 #ifdef INET6
 static void in6_getaddr(char *, int);
index b202ac4..47bd6cf 100644 (file)
 #include "inetd.h"
 
 void           chargen_dg(int, struct servtab *);
-void           chargen_stream(int, struct servtab *);
+void           chargen_stream(int, struct servtab *) __dead2;
 void           daytime_dg(int, struct servtab *);
 void           daytime_stream(int, struct servtab *);
 void           discard_dg(int, struct servtab *);
-void           discard_stream(int, struct servtab *);
+void           discard_stream(int, struct servtab *) __dead2;
 void           echo_dg(int, struct servtab *);
-void           echo_stream(int, struct servtab *);
+void           echo_stream(int, struct servtab *) __dead2;
 static int     get_line(int, char *, int);
-void           iderror(int, int, int, const char *);
-void           ident_stream(int, struct servtab *);
+void           iderror(int, int, int, const char *) __dead2;
+void           ident_stream(int, struct servtab *) __dead2;
 void           initring(void);
 unsigned long  machtime(void);
 void           machtime_dg(int, struct servtab *);
index 1c38594..c2bbbd5 100644 (file)
@@ -263,7 +263,7 @@ void         delay(int _millisec);
 void    displayq(struct printer *_pp, int _format);
 void    dump(const char *_nfile, const char *_datafile, int _copies);
 void    fatal(const struct printer *_pp, const char *_msg, ...)
-           __printflike(2, 3);
+           __dead2 __printflike(2, 3);
 int     firstprinter(struct printer *_pp, int *_error);
 void    free_printer(struct printer *_pp);
 void    free_request(struct request *_rp);
@@ -288,7 +288,7 @@ const
 char   *pcaperr(int _error);
 void    prank(int _n);
 void    process(const struct printer *_pp, char *_file);
-void    rmjob(const char *_printer);
+void    rmjob(const char *_printer) __dead2;
 void    rmremote(const struct printer *_pp);
 void    setprintcap(char *_newfile);
 int     set_qstate(int _action, const char *_lfname);
index 8d88ee9..a68f3d8 100644 (file)
@@ -30,7 +30,6 @@
  *      @(#)extern.h   8.1 (Berkeley) 6/6/93
  *
  * $FreeBSD: src/usr.sbin/lpr/lpc/extern.h,v 1.3.6.8 2002/07/26 03:12:07 gad Exp $
- * $DragonFly: src/usr.sbin/lpr/lpc/extern.h,v 1.2 2003/06/17 04:29:56 dillon Exp $
  */
 
 
@@ -56,7 +55,7 @@ void   generic(void (*_specificrtn)(struct printer *_pp), int _cmdopts,
            void (*_initcmd)(int _argc, char *_argv[]),
            int _argc, char *_argv[]);
 void    help(int _argc, char *_argv[]);
-void    quit(int _argc, char *_argv[]);
+void    quit(int _argc, char *_argv[]) __dead2;
 void    restart_q(struct printer *_pp);
 void    setstatus_gi(int _argc, char *_argv[]);
 void    setstatus_q(struct printer *_pp);
index b574fd3..c51221d 100644 (file)
@@ -39,8 +39,8 @@ struct        printer;
 struct termios;
 
 __BEGIN_DECLS
-void    printjob(struct printer *_pp);
-void    startprinting(const char *_printer);
+void    printjob(struct printer *_pp) __dead2;
+void    startprinting(const char *_printer) __dead2;
 void    recvjob(const char *_printer);
 int     msearch(char *_str, struct termios *_ip);
 __END_DECLS
index 4a6ad5f..bbc0160 100644 (file)
@@ -64,9 +64,9 @@ static int s;
 #define QUERY_RESPONSE_INTERVAL 10000
 
 static void make_msg(int, struct in6_addr *, u_int);
-static void usage(void);
+static void usage(void) __dead2;
 static void dump(int);
-static void quit(int);
+static void quit(int) __dead2;
 
 int
 main(int argc, char *argv[])
index eeb528b..814c348 100644 (file)
@@ -326,7 +326,7 @@ int                 mtrace_callback(int, u_char *, int, struct igmp *,
 int                    send_recv(u_int32 dst, int type, int code,
                                        int tries, struct resp_buf *save,
                                        callback_t callback);
-void                   passive_mode(void);
+void                   passive_mode(void) __dead2;
 char *                 print_host(u_int32 addr);
 char *                 print_host2(u_int32 addr1, u_int32 addr2);
 void                   print_trace(int idx, struct resp_buf *buf,
index a82e707..3967f63 100644 (file)
@@ -43,7 +43,7 @@
 
 int lineno = 1;
 
-void yyerror(const char *);
+void yyerror(const char *) __dead2;
 
 static void
 update_lineno(const char *cp)
index f7515d4..4d8a778 100644 (file)
@@ -135,7 +135,7 @@ static struct in6_nbrinfo *getnbrinfo(struct in6_addr *addr,
                                           int ifindex, int);
 static char *ether_str(struct sockaddr_dl *);
 int ndp_ether_aton(char *, u_char *);
-void usage(void);
+void usage(void) __dead2;
 int rtmsg(int);
 void ifinfo(int, char **);
 void rtrlist(void);
index 37fa0b1..5e5b6f2 100644 (file)
@@ -58,7 +58,7 @@
 #include "pfctl_parser.h"
 #include "pfctl.h"
 
-void    usage(void);
+void    usage(void) __dead2;
 int     pfctl_enable(int, int);
 int     pfctl_disable(int, int);
 int     pfctl_clear_stats(int, int);
index f6f35f6..bb496c6 100644 (file)
@@ -91,7 +91,7 @@ void  sig_alrm(int);
 void  sig_usr1(int);
 void  sig_close(int);
 void  sig_hup(int);
-void  usage(void);
+void  usage(void) __dead2;
 
 static int try_reset_dump(int);
 
index 7bc8c7f..624d0c5 100644 (file)
@@ -26,8 +26,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.sbin/ppp/main.h,v 1.11.2.1 2001/06/13 21:56:36 brian Exp $
- * $DragonFly: src/usr.sbin/ppp/main.h,v 1.2 2003/06/17 04:30:00 dillon Exp $
  */
 
 extern void Cleanup(void);
-extern void AbortProgram(int);
+extern void AbortProgram(int) __dead2;
index 613b6f9..4469d7c 100644 (file)
@@ -29,7 +29,6 @@
  * @(#) Copyright (c) 1991, 1993, 1994 The Regents of the University of California.  All rights reserved.
  * @(#)pwd_mkdb.c      8.5 (Berkeley) 4/20/94
  * $FreeBSD: src/usr.sbin/pwd_mkdb/pwd_mkdb.c,v 1.51 2005/06/15 10:13:04 dd Exp $
- * $DragonFly: src/usr.sbin/pwd_mkdb/pwd_mkdb.c,v 1.5 2005/12/05 02:40:27 swildner Exp $
  */
 
 #include <sys/param.h>
@@ -76,7 +75,7 @@ static int is_comment;        /* flag for comments */
 static char line[LINE_MAX];
 
 void   cleanup(void);
-void   error(const char *);
+void   error(const char *) __dead2;
 void   cp(char *, char *, mode_t mode);
 void   mv(char *, char *);
 int    scan(FILE *, struct passwd *);
index c5015b3..4379e4d 100644 (file)
@@ -221,7 +221,7 @@ static void ifdump(int);
 static void ifdump0(FILE *, const struct ifc *);
 static void rtdump(int);
 static void rt_entry(struct rt_msghdr *, int);
-static void rtdexit(void);
+static void rtdexit(void) __dead2;
 static void riprequest(struct ifc *, struct netinfo6 *, int,
        struct sockaddr_in6 *);
 static void ripflush(struct ifc *, struct sockaddr_in6 *);
@@ -243,7 +243,7 @@ static struct ifac *ifa_match(const struct ifc *, const struct in6_addr *, int);
 static struct in6_addr *plen2mask(int);
 static struct riprt *rtsearch(struct netinfo6 *, struct riprt **);
 static int ripinterval(int);
-static void fatal(const char *, ...) __printflike(1, 2);
+static void fatal(const char *, ...) __dead2 __printflike(1, 2);
 static void trace(int, const char *, ...) __printflike(2, 3);
 static void tracet(int, const char *, ...) __printflike(2, 3);
 static struct ifc *ifc_find(char *);
index 78c8888..e3f884b 100644 (file)
@@ -31,7 +31,6 @@
  *
  * $NetBSD: lockd.c,v 1.7 2000/08/12 18:08:44 thorpej Exp $
  * $FreeBSD: src/usr.sbin/rpc.lockd/lockd.c,v 1.6 2001/03/19 12:50:09 alfred Exp $
- * $DragonFly: src/usr.sbin/rpc.lockd/lockd.c,v 1.5 2005/11/25 00:32:49 swildner Exp $
  */
 
 /*
@@ -84,9 +83,9 @@ void  nlm_prog_0(struct svc_req *, SVCXPRT *);
 void   nlm_prog_1(struct svc_req *, SVCXPRT *);
 void   nlm_prog_3(struct svc_req *, SVCXPRT *);
 void   nlm_prog_4(struct svc_req *, SVCXPRT *);
-void   out_of_mem(void);
+void   out_of_mem(void) __dead2;
 
-static void    usage(void);
+static void    usage(void) __dead2;
 
 void   sigalarm_handler(void);
 
index 131706d..0842d56 100644 (file)
@@ -174,7 +174,7 @@ reaper(int sig)
        }
 }
 
-void
+__dead2 void
 usage(void)
 {
        fprintf(stderr, "rpc.ypupdatedd [-p path]\n");
index 7da9e15..8bfa10b 100644 (file)
@@ -182,7 +182,7 @@ reaper(int sig)
        return;
 }
 
-void
+__dead2 void
 usage(void)
 {
        fprintf(stderr, "usage: rpc.ypxfrd [-p path]\n");
index d434fe9..525420e 100644 (file)
@@ -112,9 +112,9 @@ void *rpcbproc_taddr2uaddr_com(void *, struct svc_req *, SVCXPRT *,
 int create_rmtcall_fd(struct netconfig *);
 void rpcbproc_callit_com(struct svc_req *, SVCXPRT *, rpcvers_t,
                              rpcvers_t);
-void my_svc_run(void);
+void my_svc_run(void) __dead2;
 
-void rpcbind_abort(void);
+void rpcbind_abort(void) __dead2;
 void reap(int);
 void toggle_verboselog(int);
 
index ba08350..90cd099 100644 (file)
@@ -142,11 +142,11 @@ static int         configure(int);
 static void     getboottime(void);
 static void     send_host_information(void);
 static void     hup(int);
-static void     quit(const char *, int);
+static void     quit(const char *, int) __dead2;
 static void     rt_xaddrs(caddr_t, caddr_t, struct rt_addrinfo *);
 static int      verify(char *, int);
 static void     handleread(int);
-static void     usage(void);
+static void     usage(void) __dead2;
 static void     timeadd(struct timeval *, time_t, struct timeval *);
 
 #ifdef DEBUG
index 898d8ef..095ba1d 100644 (file)
@@ -74,7 +74,7 @@ struct sdlim_t {
        TAILQ_HEAD(, limits_t)  limits;
 };
 
-static void             usage(void);
+static void             usage(void) __dead2;
 static void             create(void);
 static struct sdlim_t  *create_sdlim(struct sensordev *);
 static void             destroy_sdlim(struct sdlim_t *);
index 468eb99..55e3445 100644 (file)
 
 #include "libpfkey.h"
 
-void Usage(void);
+void Usage(void) __dead2;
 int get_supported(void);
 void sendkeyshort(u_int);
-void promisc(void);
+void promisc(void) __dead2;
 int sendkeymsg(void);
 int postproc(struct sadb_msg *, int);
 const char *numstr(int);
index ede478a..1b9ed72 100644 (file)
@@ -75,7 +75,7 @@ extern u_char m_buf[BUFSIZ];
 extern u_int m_len;
 extern int f_debug;
 
-void yyfatal(const char *s);
+void yyfatal(const char *s) __dead2;
 void yyerror(const char *s);
 extern void parse_init(void);
 int parse(FILE **);
index 19620d2..64291e9 100644 (file)
@@ -45,7 +45,7 @@
 
 #define MAX_CLIENTS    256
 
-void usage(void);
+void usage(void) __dead2;
 
 
 struct pollfd poll_fds[MAX_CLIENTS];
index 6abf3fb..42e70f0 100644 (file)
@@ -61,7 +61,7 @@ void   ignoreack(void);
 int     in_cksum(u_short *, int);
 void    lookformaster(struct netinfo *);
 void    makeslave(struct netinfo *);
-void    master(void);
+void    master(void) __dead2;
 void    masterack(void);
 void    masterup(struct netinfo *);
 int     measure(u_long, u_long, char *, struct sockaddr_in *, int);
@@ -73,7 +73,7 @@ void   print(struct tsp *, struct sockaddr_in *);
 void    prthp(clock_t);
 void    rmnetmachs(struct netinfo *);
 void    setstatus(void);
-void    slave(void);
+void    slave(void) __dead2;
 void    slaveack(void);
 void    spreadtime(void);
 void    suppress(struct sockaddr_in *, char *, struct netinfo *);
index 7d90e3e..1a463c0 100644 (file)
@@ -27,7 +27,6 @@
  * SUCH DAMAGE.
  *
  *     @(#)extern.h    8.1 (Berkeley) 6/6/93
- * $DragonFly: src/usr.sbin/timed/timedc/extern.h,v 1.3 2003/11/03 19:31:43 eirikn Exp $
  */
 
 #if __STDC__
@@ -40,10 +39,10 @@ void        bytehostorder(struct tsp *);
 void   bytenetorder(struct tsp *);
 void   clockdiff(int, char *[]);
 void   help(int, char *[]);
-void   intr(int);
+void   intr(int) __dead2;
 void   makeargv(void);
 void   msite(int, char *[]);
 int    priv_resources(void);
-void   quit(void);
+void   quit(void) __dead2;
 void   testing(int, char *[]);
 void   tracing(int, char *[]);
index be5497d..ca74ae4 100644 (file)
@@ -287,7 +287,7 @@ static const char *pr_type(u_int8_t);
 static void print(u_char *, int, struct sockaddr_in *);
 static char *inetname(struct in_addr);
 static u_short in_cksum(u_short *, int);
-static void usage(void);
+static void usage(void) __dead2;
 
 static int s;                  /* receive (icmp) socket file descriptor */
 static int sndsock;            /* send (udp) socket file descriptor */
index 24c6e23..86ee12c 100644 (file)
@@ -309,7 +309,7 @@ static char *pr_type(int);
 static int     packet_ok(struct msghdr *, int, int);
 static void    print(struct msghdr *, int);
 static const char *inetname(struct sockaddr *);
-static void    usage(void);
+static void    usage(void) __dead2;
 
 static int rcvsock;            /* receive (icmp) socket file descriptor */
 static int sndsock;            /* send (udp) socket file descriptor */
index 1722c39..b661d54 100644 (file)
@@ -34,7 +34,7 @@
 
 static volatile sig_atomic_t   quit = 0;
 
-static void    usage(void);
+static void    usage(void) __dead2;
 static void    sighdlr(int);
 
 static void
index ec9d3d6..054f100 100644 (file)
@@ -27,7 +27,6 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.sbin/ypbind/ypbind.c,v 1.40 2004/10/17 19:33:33 stefanf Exp $
- * $DragonFly: src/usr.sbin/ypbind/ypbind.c,v 1.6 2005/11/24 22:23:02 swildner Exp $
  */
 
 #include <sys/param.h>
@@ -101,7 +100,7 @@ int ping(struct _dom_binding *);
 int    tell_parent(char *, struct sockaddr_in *);
 void   handle_children(struct _dom_binding *);
 void   reaper(int);
-void   terminate(int);
+void   terminate(int) __dead2;
 void   yp_restricted_mode(char *);
 int    verify(struct in_addr);