]> git.ipfire.org Git - thirdparty/grub.git/commitdiff
net/net: Fix incorrect condition for calling grub_net_tcp_retransmit()
authorGlenn Washburn <development@efficientek.com>
Thu, 12 May 2022 02:44:01 +0000 (21:44 -0500)
committerDaniel Kiper <daniel.kiper@oracle.com>
Tue, 24 May 2022 12:37:24 +0000 (14:37 +0200)
The commit 848724273e4 (net/net: Avoid unnecessary calls to
grub_net_tcp_retransmit()) needs to have its condition inverted to avoid
unnecessary calls to grub_net_tcp_retransmit(). As it is, it creates many
unnecessary calls and does not call grub_net_tcp_retransmit() when needed.
The call to grub_net_tcp_retransmit() should only be made when
grub_net_cards does _not_ equal NULL, meaning that there are potentially
network cards that need TCP retransmission.

Fixes: 848724273e4 (net/net: Avoid unnecessary calls to grub_net_tcp_retransmit())
Signed-off-by: Glenn Washburn <development@efficientek.com>
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
grub-core/net/net.c

index 88ea49fee8d5dc0fc338921bba6d5bec99a858d1..2b67715235ec787039485595cd9209315f38dbbb 100644 (file)
@@ -1634,7 +1634,7 @@ grub_net_poll_cards_idle_real (void)
        || ctime >= card->last_poll + card->idle_poll_delay_ms)
       receive_packets (card, 0);
   }
-  if (grub_net_cards == NULL)
+  if (grub_net_cards != NULL)
     grub_net_tcp_retransmit ();
 }