Revision: 18799
          http://sourceforge.net/p/edk2/code/18799
Author:   vanjeff
Date:     2015-11-16 06:59:13 +0000 (Mon, 16 Nov 2015)
Log Message:
-----------
MdeModulePkg: reset DHCP child when leaving PXE LoadFile.

The DHCP4 can have only one configured child instance so we need to reset
the DHCP4 child when leaving PXE driver's LoadFile() function, otherwise the
other programs which also need to use DHCP4 (like HTTP boot) will be impacted.

(Sync patch r18615 from main trunk.)

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <[email protected]>
Reviewed-by: Sriram Subramanian <[email protected]>
Reviewed-by: Ye Ting <[email protected]>

Revision Links:
--------------
    http://sourceforge.net/p/edk2/code/18615

Modified Paths:
--------------
    branches/UDK2015/MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcImpl.c

Modified: 
branches/UDK2015/MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcImpl.c
===================================================================
--- branches/UDK2015/MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcImpl.c    
2015-11-16 06:58:15 UTC (rev 18798)
+++ branches/UDK2015/MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcImpl.c    
2015-11-16 06:59:13 UTC (rev 18799)
@@ -2902,9 +2902,14 @@
   //
   if (Status == EFI_SUCCESS) {
     //
+    // The DHCP4 can have only one configured child instance so we need to stop
+    // reset the DHCP4 child before we return. Otherwise the other programs 
which 
+    // also need to use DHCP4 will be impacted.
     // The functionality of PXE Base Code protocol will not be stopped,
     // when downloading is successfully.
     //
+    Private->Dhcp4->Stop (Private->Dhcp4);
+    Private->Dhcp4->Configure (Private->Dhcp4, NULL);
     return EFI_SUCCESS;
 
   } else if (Status == EFI_BUFFER_TOO_SMALL) {


------------------------------------------------------------------------------
Presto, an open source distributed SQL query engine for big data, initially
developed by Facebook, enables you to easily query your data on Hadoop in a 
more interactive manner. Teradata is also now providing full enterprise
support for Presto. Download a free open source copy now.
http://pubads.g.doubleclick.net/gampad/clk?id=250295911&iu=/4140
_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to