[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...

[cgutman]

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/oskittcp/interface.c?rev=52505&r1=52504&r2=52505&view=diff
==============================================================================
--- 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;