Reviewed-by: Maciej Rabeda <maciej.rab...@linux.intel.com>

On 28-Sep-20 08:29, Abner Chang wrote:
BZ #2909,
https://bugzilla.tianocore.org/show_bug.cgi?id=2909

When Http->Configure() is invoked with HttpConfigData set to
NULL to reset the EFI HTTP instance, TLS child instance is
destroyed but HttpInstance->TlsChildHandle is not set to
NULL. After reconfiguring HTTP through Http->Configure()
and sending the HTTP request to HTTPS URL, TLS child
instance is not recreated because
HttpInstance->TlsChildHandle is not NULL.

Signed-off-by: Abner Chang <abner.ch...@hpe.com>

Cc: Maciej Rabeda <maciej.rab...@linux.intel.com>
Cc: Wu Jiaxin <jiaxin...@intel.com>
Cc: Fu Siyuan <siyuan...@intel.com>
Cc: Nickle Wang <nickle.w...@hpe.com>
---
  NetworkPkg/HttpDxe/HttpProto.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/NetworkPkg/HttpDxe/HttpProto.c b/NetworkPkg/HttpDxe/HttpProto.c
index 3c7c6ff9f0c..afc7db5a726 100644
--- a/NetworkPkg/HttpDxe/HttpProto.c
+++ b/NetworkPkg/HttpDxe/HttpProto.c
@@ -873,6 +873,7 @@ HttpCleanProtocol (
      // Destroy the TLS instance.
      //
      HttpInstance->TlsSb->DestroyChild (HttpInstance->TlsSb, 
HttpInstance->TlsChildHandle);
+    HttpInstance->TlsChildHandle = NULL;
    }
if (HttpInstance->Tcp4ChildHandle != NULL) {



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#65727): https://edk2.groups.io/g/devel/message/65727
Mute This Topic: https://groups.io/mt/77169386/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to