Re: [PATCH] NetworkPkg/HttpDxe: Clear TlsChildHandle during cleanup

Maciej Rabeda

Hi Scott,

Thanks for submitting the patch - I am about to approve the patch, however - a couple of small remarks to the commit message.
1. Please remove the "From:" line
2. Please add "Cc:" lines before "Signed-off-by". Cc people are added based on appropriate EDK2 package maintainer & reviewer list:
Example of a patch from NetworkPkg:


On 08-Sep-20 06:50, D Scott Phillips wrote:
On Monday, September 7, 2020 4:33 AM, Laszlo Ersek <lersek@...> wrote:

Hi Scott,


On 09/05/20 03:15, D Scott Phillips wrote:

From: D Scott Phillips d.scott.phillips@...
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.

Signed-off-by: D Scott Phillips d.scott.phillips@...


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) {
thanks a lot for tracking this down!

I've reopened BZ#1917, and linked your patch email in a new comment.

But, I'd also like to assign the BZ to you, if that's OK with you. Can
you please register in the TianoCore bugzilla instance for that?
Certainly, account created and assignment taken. Thanks Laszlo.


Join to automatically receive all group messages.