We don't need to break after calling err(3) because this function will
not return.  This diff produces no binary change.

First diff from Michael W. Bombardieri.

ok?

Index: server.c
===================================================================
RCS file: /cvs/src/usr.sbin/pppoe/server.c,v
retrieving revision 1.13
diff -u -p -r1.13 server.c
--- server.c    3 Sep 2007 14:26:54 -0000       1.13
+++ server.c    11 Oct 2011 12:57:40 -0000
@@ -112,7 +112,7 @@ server_mode(int bpffd, u_int8_t *sysname
                        if (errno == EINTR)
                                continue;
                        err(EX_IOERR, "select");
-                       break;
+                       /* NOTREACHED */
                }
                if (n == 0)
                        continue;

Reply via email to