From: Peter Hater <7element@mail.bg>
Date: Thu, 25 May 2017 12:34:33 +0000 (+0000)
Subject: [WS2_32] Perform parameters check on bind. ROSTESTS-278
X-Git-Tag: ReactOS-0.4.6~641
X-Git-Url: https://git.reactos.org/?p=reactos.git;a=commitdiff_plain;h=0e96026122952e1e7fa15296e6fd45c391a28003

[WS2_32] Perform parameters check on bind. ROSTESTS-278

svn path=/trunk/; revision=74649
---

diff --git a/reactos/dll/win32/ws2_32/src/socklife.c b/reactos/dll/win32/ws2_32/src/socklife.c
index 254466afd52..86a12c6017a 100644
--- a/reactos/dll/win32/ws2_32/src/socklife.c
+++ b/reactos/dll/win32/ws2_32/src/socklife.c
@@ -48,19 +48,27 @@ bind(IN SOCKET s,
         /* Get the Socket Context */
         if ((Socket = WsSockGetSocket(s)))
         {
-            /* Make the call */
-            Status = Socket->Provider->Service.lpWSPBind(s,
-                                                         name,
-                                                         namelen,
-                                                         &ErrorCode);
-            /* Deference the Socket Context */
-            WsSockDereference(Socket);
+            if (name && (namelen >= sizeof(struct sockaddr)))
+            {
+                /* Make the call */
+                Status = Socket->Provider->Service.lpWSPBind(s,
+                                                             name,
+                                                             namelen,
+                                                             &ErrorCode);
+                /* Deference the Socket Context */
+                WsSockDereference(Socket);
 
-            /* Return Provider Value */
-            if (Status == ERROR_SUCCESS) return Status;
+                /* Return Provider Value */
+                if (Status == ERROR_SUCCESS) return Status;
 
-            /* If everything seemed fine, then the WSP call failed itself */
-            if (ErrorCode == NO_ERROR) ErrorCode = WSASYSCALLFAILURE;
+                /* If everything seemed fine, then the WSP call failed itself */
+                if (ErrorCode == NO_ERROR) ErrorCode = WSASYSCALLFAILURE;
+            }
+            else
+            {
+                /* name or namelen not valid */
+                ErrorCode = WSAEFAULT;
+            }
         }
         else
         {