[ros-diffs] [cgutman] 36071: - Make sure we set a failure status when we fail so we don't complete with NDIS_STATUS_SUCCESS when we really failed

[cgutman at svn.reactos.org]

Author: cgutman
Date: Mon Sep  8 18:42:14 2008
New Revision: 36071

URL: http://svn.reactos.org/svn/reactos?rev=36071&view=rev
Log:
 - Make sure we set a failure status when we fail so we don't complete with NDIS_STATUS_SUCCESS when we really failed

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

Modified: branches/aicom-network-fixes/drivers/network/ndis/ndis/stubs.c
URL: http://svn.reactos.org/svn/reactos/branches/aicom-network-fixes/drivers/network/ndis/ndis/stubs.c?rev=36071&r1=36070&r2=36071&view=diff
==============================================================================
--- branches/aicom-network-fixes/drivers/network/ndis/ndis/stubs.c [iso-8859-1] (original)
+++ branches/aicom-network-fixes/drivers/network/ndis/ndis/stubs.c [iso-8859-1] Mon Sep  8 18:42:14 2008
@@ -301,7 +301,10 @@
   memmove ( FullFileName.Buffer, NDIS_FILE_FOLDER, FullFileName.Length );
   *Status = RtlAppendUnicodeStringToString ( &FullFileName, FileName );
   if ( !NT_SUCCESS(*Status) )
+  {
+    *Status = NDIS_STATUS_FAILURE;
     goto cleanup;
+  }
 
   InitializeObjectAttributes ( &ObjectAttributes,
     &FullFileName,
@@ -321,6 +324,11 @@
     FILE_SYNCHRONOUS_IO_NONALERT, // ULONG CreateOptions
     0, // PVOID EaBuffer
     0 ); // ULONG EaLength
+  
+  if ( !NT_SUCCESS(*Status) )
+  {
+    *Status = NDIS_STATUS_FAILURE;
+  }
 
 cleanup:
   if ( FullFileName.Buffer != NULL )