From 28a0cf4aa03e0a43d63492844ef3f3eee9da84a5 Mon Sep 17 00:00:00 2001
From: patacongo <patacongo@42af7a65-404d-4744-a932-0658087f49c3>
Date: Sun, 20 Jan 2013 00:41:33 +0000
Subject: Yet another repair for the previouis botched recvfrom() fix; Fix
 telnet driver: It needs to break out of the read loop if 0 (meaning not
 conneced) of a value < 0 (an error) is encountered.

git-svn-id: http://svn.code.sf.net/p/nuttx/code/trunk@5541 42af7a65-404d-4744-a932-0658087f49c3
---
 nuttx/net/recvfrom.c | 43 +++++++++++++++++++++++++------------------
 1 file changed, 25 insertions(+), 18 deletions(-)

(limited to 'nuttx/net')

diff --git a/nuttx/net/recvfrom.c b/nuttx/net/recvfrom.c
index ac8065f81..a1a6742c5 100644
--- a/nuttx/net/recvfrom.c
+++ b/nuttx/net/recvfrom.c
@@ -571,40 +571,47 @@ static uint16_t recvfrom_tcpinterrupt(struct uip_driver_s *dev, void *conn,
            *     gracefully disconnected
            *   _SF_CONNECTED==0 && _SF_CLOSED==0 - the socket was
            *     rudely disconnected
-           *
-           * These flag settings are probably not necessary if
-           * CONFIG_NET_TCP_RECVDELAY == 0; in that case we know that
-           * pstate->rf_recvlen == 0 and we will always return -ENOTCONN.
            */
 
           psock = pstate->rf_sock;
           if ((flags & UIP_CLOSE) != 0)
             {
+              /* Report that the connection was gracefully closed */
+
               psock->s_flags &= ~_SF_CONNECTED;
               psock->s_flags |= _SF_CLOSED;
+
+              /* This case should always return success (zero)! The value of
+               * rf_recvlen, if zero, will indicate that the connection was
+               * gracefully closed.
+               */
+
+              pstate->rf_result = 0;
             }
           else
             {
+              /* Report that the connection was rudely lost */
+
               psock->s_flags &= ~(_SF_CONNECTED |_SF_CLOSED);
-            }
 
-          /* If no data has been received, then return ENOTCONN.
-           * Otherwise, let this return success.  The failure will
-           * be reported the next time that recv[from]() is called.
-           */
+              /* If no data has been received, then return ENOTCONN.
+               * Otherwise, let this return success.  The failure will
+               * be reported the next time that recv[from]() is called.
+               */
 
 #if CONFIG_NET_TCP_RECVDELAY > 0
-          if (pstate->rf_recvlen > 0)
-            {
-              pstate->rf_result = 0;
-            }
-          else
-            {
-              pstate->rf_result = -ENOTCONN;
-            }
+              if (pstate->rf_recvlen > 0)
+                {
+                  pstate->rf_result = 0;
+                }
+              else
+                {
+                  pstate->rf_result = -ENOTCONN;
+                }
 #else
-          pstate->rf_result = -ENOTCONN;
+              pstate->rf_result = -ENOTCONN;
 #endif
+            }
 
           /* Wake up the waiting thread */
 
-- 
cgit v1.2.3