ShellPkg/Ping: fix loss of first packet

Issue:
Reply for first ping packet was getting dropped.

Cause:
Sometimes reply message comes even before trasmit
function returns, hence missing 1st reply

Fix:
Prepare the TxList before calling Transmit function.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Meenakshi Aggarwal <meenakshi.aggarwal@nxp.com>
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com>
Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>

diff --git a/ShellPkg/Library/UefiShellNetwork1CommandsLib/Ping.c b/ShellPkg/Library/UefiShellNetwork1CommandsLib/Ping.c
index bec9535a8b436338c91e94290fde58a900e43804..46ba701fc544a754c9ddeed158111c0f8f4781c7 100644 (file)
--- a/ShellPkg/Library/UefiShellNetwork1CommandsLib/Ping.c
+++ b/ShellPkg/Library/UefiShellNetwork1CommandsLib/Ping.c
@@ -784,14 +784,17 @@ PingSendEchoRequest (
   }\r
 \r
   ASSERT(Private->ProtocolPointers.Transmit != NULL);\r
+\r
+  InsertTailList (&Private->TxList, &TxInfo->Link);\r
+\r
   Status = Private->ProtocolPointers.Transmit (Private->IpProtocol, TxInfo->Token);\r
 \r
   if (EFI_ERROR (Status)) {\r
+    RemoveEntryList (&TxInfo->Link);\r
     PingDestroyTxInfo (TxInfo, Private->IpChoice);\r
     return Status;\r
   }\r
 \r
-  InsertTailList (&Private->TxList, &TxInfo->Link);\r
   Private->TxCount++;\r
 \r
   return EFI_SUCCESS;\r