File 0027-net-http-Do-not-tear-down-socket-if-it-s-a... of Package grub2

Overview Repositories Revisions Requests Users Attributes Meta

File 0027-net-http-Do-not-tear-down-socket-if-it-s-already-bee.patch of Package grub2 (Revision fa734166bf63e388fdefb63b9f1e0680)

Currently displaying revision fa734166bf63e388fdefb63b9f1e0680 , Show latest

From cc232d3abd48c69a325f897dc84d2f5cd14acf2c Mon Sep 17 00:00:00 2001
From: Daniel Axtens <dja@axtens.net>
Date: Tue, 1 Mar 2022 23:14:15 +1100
Subject: [PATCH 27/33] net/http: Do not tear down socket if it's already been
 torn down

It's possible for data->sock to get torn down in tcp error handling.
If we unconditionally tear it down again we will end up doing writes
to an offset of the NULL pointer when we go to tear it down again.

Detect if it has been torn down and don't do it again.

Signed-off-by: Daniel Axtens <dja@axtens.net>
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
---
 grub-core/net/http.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/grub-core/net/http.c b/grub-core/net/http.c
index 5004ecfee4..7dfdb2372e 100644
--- a/grub-core/net/http.c
+++ b/grub-core/net/http.c
@@ -425,7 +425,7 @@ http_establish (struct grub_file *file, grub_off_t offset, int initial)
       return err;
     }
 
-  for (i = 0; !data->headers_recv && i < 100; i++)
+  for (i = 0; data->sock && !data->headers_recv && i < 100; i++)
     {
       grub_net_tcp_retransmit ();
       grub_net_poll_cards (300, &data->headers_recv);
@@ -433,7 +433,8 @@ http_establish (struct grub_file *file, grub_off_t offset, int initial)
 
   if (!data->headers_recv)
     {
-      grub_net_tcp_close (data->sock, GRUB_NET_TCP_ABORT);
+      if (data->sock)
+        grub_net_tcp_close (data->sock, GRUB_NET_TCP_ABORT);
       if (data->err)
 	{
 	  char *str = data->errmsg;
-- 
2.34.1

Places

File 0027-net-http-Do-not-tear-down-socket-if-it-s-already-bee.patch of Package grub2 (Revision fa734166bf63e388fdefb63b9f1e0680)

Places