File 0375-e1000-fail-early-for-evil-descripto.patch of Package qemu-linux-user.20756

Overview Repositories Revisions Requests Users Attributes Meta

File 0375-e1000-fail-early-for-evil-descripto.patch of Package qemu-linux-user.20756

From: Jason Wang <jasowang@redhat.com>
Date: Wed, 24 Feb 2021 13:45:28 +0800
Subject: e1000: fail early for evil descriptor

Git-commit: 3de46e6fc489c52c9431a8a832ad8170a7569bd8
References: bsc#1182577, CVE-2021-20257

During procss_tx_desc(), driver can try to chain data descriptor with
legacy descriptor, when will lead underflow for the following
calculation in process_tx_desc() for bytes:

if (tp->size + bytes > msh)
                bytes = msh - tp->size;

This will lead a infinite loop. So check and fail early if tp->size if
greater or equal to msh.

Reported-by: Alexander Bulekov <alxndr@bu.edu>
Reported-by: Cheolwoo Myung <cwmyung@snu.ac.kr>
Reported-by: Ruhr-University Bochum <bugs-syssec@rub.de>
Cc: Prasad J Pandit <ppandit@redhat.com>
Cc: qemu-stable@nongnu.org
Signed-off-by: Jason Wang <jasowang@redhat.com>
Signed-off-by: Bruce Rogers <brogers@suse.com>
---
 hw/net/e1000.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/hw/net/e1000.c b/hw/net/e1000.c
index 8e79b550e6bdb17fa9c5f6aceb11..49e6a5f80ac38470eda1b4f990d8 100644
--- a/hw/net/e1000.c
+++ b/hw/net/e1000.c
@@ -817,6 +817,9 @@ process_tx_desc(E1000State *s, struct e1000_tx_desc *dp)
         msh = tp->hdr_len + tp->mss;
         do {
             bytes = split_size;
+            if (tp->size >= msh) {
+                goto eop;
+            }
             if (tp->size + bytes > msh)
                 bytes = msh - tp->size;
 
@@ -844,6 +847,7 @@ process_tx_desc(E1000State *s, struct e1000_tx_desc *dp)
         tp->size += split_size;
     }
 
+eop:
     if (!(txd_lower & E1000_TXD_CMD_EOP))
         return;
     if (!(tp->tse && tp->cptse && tp->size < tp->hdr_len)) {

Places

File 0375-e1000-fail-early-for-evil-descripto.patch of Package qemu-linux-user.20756

Places