NetworkPkg/HttpDxe: Decofigure Tcp6 before reconfiguring

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2720

Check if the state of the HTTP instance is HTTP_STATE_TCP_CONNECTED, or
HTTP_STATE_TCP_CLOSED and de-configure the Tcp6 instance before
configuring it again.

Signed-off-by: Oliver Steffen <osteffen@redhat.com>
Reviewed-by: Maciej Rabeda <maciej.rabeda@linux.intel.com>
Reviewed-by: Gerd Hoffmann <kraxel@redhat.com>

diff --git a/NetworkPkg/HttpDxe/HttpProto.c b/NetworkPkg/HttpDxe/HttpProto.c
index cd54c574044b9500348560f530ce279967e4d1d2..33ae622c3f0b21dc75e97d324f6b5ef2c35cb7e5 100644 (file)
--- a/NetworkPkg/HttpDxe/HttpProto.c
+++ b/NetworkPkg/HttpDxe/HttpProto.c
@@ -1168,6 +1168,18 @@ HttpConfigureTcp6 (
   Tcp6Option->KeepAliveInterval = HTTP_KEEP_ALIVE_INTERVAL;\r
   Tcp6Option->EnableNagle       = TRUE;\r
 \r
+  if ((HttpInstance->State == HTTP_STATE_TCP_CONNECTED) ||\r
+      (HttpInstance->State == HTTP_STATE_TCP_CLOSED))\r
+  {\r
+    Status = HttpInstance->Tcp6->Configure (HttpInstance->Tcp6, NULL);\r
+    if (EFI_ERROR (Status)) {\r
+      DEBUG ((DEBUG_ERROR, "HttpConfigureTcp6(NULL) - %r\n", Status));\r
+      return Status;\r
+    }\r
+\r
+    HttpInstance->State = HTTP_STATE_TCP_UNCONFIGED;\r
+  }\r
+\r
   Status = HttpInstance->Tcp6->Configure (HttpInstance->Tcp6, Tcp6CfgData);\r
   if (EFI_ERROR (Status)) {\r
     DEBUG ((DEBUG_ERROR, "HttpConfigureTcp6 - %r\n", Status));\r