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;