[ros-diffs] [cgutman] 52505: [OSKITTCP] - Remove socket state checks from OskitTCPRecv and OskitTCPSend - Oskit already returns ESHUTDOWN when sosend is called on a socket with SS_CANTSENDMORE so it is unneces...

Author: cgutman
Date: Sat Jul  2 22:57:30 2011
New Revision: 52505

URL: http://svn.reactos.org/svn/reactos?rev=52505&view=rev
Log:
[OSKITTCP]
- Remove socket state checks from OskitTCPRecv and OskitTCPSend
- Oskit already returns ESHUTDOWN when sosend is called on a socket with SS_CANTSENDMORE
so it is unnecessary
- The check in OskitTCPRecv broke returning success and 0 bytes on a graceful shutdown (We
got the FIN, set SS_CANTRCVMORE, signaled the connection, tried to read and failed with
ESHUTDOWN instead of succeeding with 0 bytes)

Modified:
    trunk/reactos/lib/drivers/oskittcp/oskittcp/interface.c

Modified: trunk/reactos/lib/drivers/oskittcp/oskittcp/interface.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/lib/drivers/oskittcp/oskit…
==============================================================================
--- trunk/reactos/lib/drivers/oskittcp/oskittcp/interface.c [iso-8859-1] (original)
+++ trunk/reactos/lib/drivers/oskittcp/oskittcp/interface.c [iso-8859-1] Sat Jul  2
22:57:30 2011
@@ -158,9 +158,6 @@
     
     if (!connection)
         return OSK_ESHUTDOWN;
-    
-    if (so->so_state & SS_CANTRCVMORE)
-        return OSK_ESHUTDOWN;
 
     OS_DbgPrint(OSK_MID_TRACE,
                 ("so->so_state %x\n", so->so_state));
@@ -302,9 +299,6 @@
     struct iovec iov;
 
     if (!socket)
-        return OSK_ESHUTDOWN;
-    
-    if (so->so_state & SS_CANTSENDMORE)
         return OSK_ESHUTDOWN;
 
     iov.iov_len = Len;