From 321d268b287854f14779055b3a7ca8fc28b09de1 Mon Sep 17 00:00:00 2001
From: Brian Somers <brian@FreeBSD.org>
Date: Fri, 24 Aug 2001 14:52:38 +0000
Subject: [PATCH] Understand that a return value of 0 from NgRecvMsg() means
 that the socket was closed.

This prevents erroneous ``Unexpected netgraph version'' from turning
up in the log.
---
 libexec/pppoed/pppoed.c | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/libexec/pppoed/pppoed.c b/libexec/pppoed/pppoed.c
index ce50ee416452..a3c6d375ac79 100644
--- a/libexec/pppoed/pppoed.c
+++ b/libexec/pppoed/pppoed.c
@@ -141,7 +141,7 @@ ConfigureNode(const char *prog, const char *iface, const char *provider,
 
   /* Get our list back */
   resp = (struct ng_mesg *)rbuf;
-  if (NgRecvMsg(cs, resp, sizeof rbuf, NULL) < 0) {
+  if (NgRecvMsg(cs, resp, sizeof rbuf, NULL) <= 0) {
     perror("Cannot get netgraph response");
     return EX_UNAVAILABLE;
   }
@@ -359,11 +359,17 @@ Spawn(const char *prog, const char *acname, const char *provider,
         syslog(LOG_INFO, "Waiting for a SUCCESS reply %s", path);
 
       do {
-        if (NgRecvMsg(cs, rep, sizeof msgbuf, NULL) < 0) {
+        if ((ret = NgRecvMsg(cs, rep, sizeof msgbuf, NULL) < 0)) {
           syslog(LOG_ERR, "%s: Cannot receive a message: %m", path);
           _exit(EX_OSERR);
         }
 
+        if (ret == 0) {
+          /* The socket has been closed */
+          syslog(LOG_INFO, "%s: Client timed out", path);
+          _exit(EX_TEMPFAIL);
+        }
+
         if (rep->header.version != NG_VERSION) {
           syslog(LOG_ERR, "%ld: Unexpected netgraph version, expected %ld",
                  (long)rep->header.version, (long)NG_VERSION);