From: D Scott Phillips <d.scott.phill...@amperecomputing.com> Leaving TlsChildHandle with the stale handle causes later use of https with the http instance to incorrectly skip tls reconfiguration, use the stale handle, and eventually call a garbage function pointer.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1917 Signed-off-by: D Scott Phillips <d.scott.phill...@amperecomputing.com> --- NetworkPkg/HttpDxe/HttpProto.c | 1 + 1 file changed, 1 insertion(+) diff --git a/NetworkPkg/HttpDxe/HttpProto.c b/NetworkPkg/HttpDxe/HttpProto.c index 3c7c6ff9f0..afc7db5a72 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) { -- 2.26.2 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#65082): https://edk2.groups.io/g/devel/message/65082 Mute This Topic: https://groups.io/mt/76683729/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
