[ros-diffs] [cgutman] 38539: - The caller has already built RequestConnectionInfo so don't do it again - Fixes a memory leak

cgutman at svn.reactos.org cgutman at svn.reactos.org
Sun Jan 4 03:46:06 CET 2009


Author: cgutman
Date: Sat Jan  3 20:46:05 2009
New Revision: 38539

URL: http://svn.reactos.org/svn/reactos?rev=38539&view=rev
Log:
 - The caller has already built RequestConnectionInfo so don't do it again
 - Fixes a memory leak

Modified:
    branches/aicom-network-fixes/drivers/network/afd/afd/tdi.c

Modified: branches/aicom-network-fixes/drivers/network/afd/afd/tdi.c
URL: http://svn.reactos.org/svn/reactos/branches/aicom-network-fixes/drivers/network/afd/afd/tdi.c?rev=38539&r1=38538&r2=38539&view=diff
==============================================================================
--- branches/aicom-network-fixes/drivers/network/afd/afd/tdi.c [iso-8859-1] (original)
+++ branches/aicom-network-fixes/drivers/network/afd/afd/tdi.c [iso-8859-1] Sat Jan  3 20:46:05 2009
@@ -420,21 +420,13 @@
         return STATUS_INVALID_PARAMETER;
   }
 
-  Status = TdiBuildNullConnectionInfo(RequestConnectionInfo,
-				      TDI_ADDRESS_TYPE_IP);
-  if (!NT_SUCCESS(Status))
-    return Status;
-
   *Irp = TdiBuildInternalDeviceControlIrp(TDI_LISTEN,              /* Sub function */
 					  DeviceObject,            /* Device object */
 					  ConnectionObject,        /* File object */
 					  NULL,                    /* Event */
 					  Iosb);                   /* Status */
   if (*Irp == NULL)
-    {
-	ExFreePool(*RequestConnectionInfo);
 	return STATUS_INSUFFICIENT_RESOURCES;
-    }
 
   TdiBuildListen(*Irp,                   /* IRP */
                  DeviceObject,           /* Device object */



More information about the Ros-diffs mailing list