[cgutman] 36334: - Code cleanup - Set Status to NDIS_STATUS_NOT_SUPPORTED if we don't support the parameter type

Author: cgutman
Date: Fri Sep 19 18:18:12 2008
New Revision: 36334

URL: http://svn.reactos.org/svn/reactos?rev=36334&view=rev
Log:
 - Code cleanup
 - Set Status to NDIS_STATUS_NOT_SUPPORTED if we don't support the parameter type

Modified:
    branches/aicom-network-fixes/drivers/network/ndis/ndis/config.c

Modified: branches/aicom-network-fixes/drivers/network/ndis/ndis/config.c
URL: http://svn.reactos.org/svn/reactos/branches/aicom-network-fixes/drivers/netw...
==============================================================================
--- branches/aicom-network-fixes/drivers/network/ndis/ndis/config.c [iso-8859-1] (original)
+++ branches/aicom-network-fixes/drivers/network/ndis/ndis/config.c [iso-8859-1] Fri Sep 19 18:18:12 2008
@@ -68,17 +68,6 @@
     PVOID Data;
     WCHAR Buff[25];

-    if(ParameterType != NdisParameterInteger &&
-        ParameterType != NdisParameterHexInteger &&
-        ParameterType != NdisParameterString &&
-        ParameterType != NdisParameterMultiString &&
-        ParameterType != NdisParameterBinary
-      )
-    {
-        *Status = NDIS_STATUS_NOT_SUPPORTED;
-        return;
-    }
-
     /* reset parameter type to standard reg types */
     switch(ParameterType)
     {
@@ -118,7 +107,7 @@
             break;

default:
-            *Status = NDIS_STATUS_FAILURE;
+            *Status = NDIS_STATUS_NOT_SUPPORTED;
             return;
     }

@@ -342,6 +331,7 @@
       )
     {
         NDIS_DbgPrint(MID_TRACE,("unsupported parameter type\n"));
+        *Status = NDIS_STATUS_NOT_SUPPORTED;
         return;
     }