net: eepro100: validate various address valuesi(CVE-2021-20255)
fix CVE-2021-20255 patch link: https://lists.gnu.org/archive/html/qemu-devel/2021-02/msg06098.html fix CVE-2021-20255, sync patch from ostms platform. Signed-off-by: zhouli57 <zhouli57@huawei.com> Signed-off-by: Yan Wang <wangyan122@huawei.com> Signed-off-by: imxcc <xingchaochao@huawei.com> (cherry picked from commit 3adeb3cd9d5cf3c2d32b4001e5d373779517b85c)
This commit is contained in:
parent
be182a9172
commit
85015397cd
58
net-eepro100-validate-various-address-valuesi-CVE-20.patch
Normal file
58
net-eepro100-validate-various-address-valuesi-CVE-20.patch
Normal file
@ -0,0 +1,58 @@
|
||||
From 5db012b1116d21c64da88ad206b3589ddf5f219b Mon Sep 17 00:00:00 2001
|
||||
From: zhouli57 <zhouli57@huawei.com>
|
||||
Date: Sat, 18 Dec 2021 09:39:57 +0800
|
||||
Subject: [PATCH] net: eepro100: validate various address
|
||||
valuesi(CVE-2021-20255)
|
||||
|
||||
fix CVE-2021-20255
|
||||
|
||||
patch link: https://lists.gnu.org/archive/html/qemu-devel/2021-02/msg06098.html
|
||||
|
||||
fix CVE-2021-20255, sync patch from ostms platform.
|
||||
|
||||
Signed-off-by: zhouli57 <zhouli57@huawei.com>
|
||||
Signed-off-by: Yan Wang <wangyan122@huawei.com>
|
||||
---
|
||||
hw/net/eepro100.c | 12 ++++++++++++
|
||||
1 file changed, 12 insertions(+)
|
||||
|
||||
diff --git a/hw/net/eepro100.c b/hw/net/eepro100.c
|
||||
index 16e95ef9cc..2474cf3dc2 100644
|
||||
--- a/hw/net/eepro100.c
|
||||
+++ b/hw/net/eepro100.c
|
||||
@@ -279,6 +279,9 @@ typedef struct {
|
||||
/* Quasi static device properties (no need to save them). */
|
||||
uint16_t stats_size;
|
||||
bool has_extended_tcb_support;
|
||||
+
|
||||
+ /* Flag to avoid recursions. */
|
||||
+ bool busy;
|
||||
} EEPRO100State;
|
||||
|
||||
/* Word indices in EEPROM. */
|
||||
@@ -837,6 +840,14 @@ static void action_command(EEPRO100State *s)
|
||||
Therefore we limit the number of iterations. */
|
||||
unsigned max_loop_count = 16;
|
||||
|
||||
+ if (s->busy) {
|
||||
+ /* Prevent recursions. */
|
||||
+ logout("recursion in %s:%u\n", __FILE__, __LINE__);
|
||||
+ return;
|
||||
+ }
|
||||
+
|
||||
+ s->busy = true;
|
||||
+
|
||||
for (;;) {
|
||||
bool bit_el;
|
||||
bool bit_s;
|
||||
@@ -933,6 +944,7 @@ static void action_command(EEPRO100State *s)
|
||||
}
|
||||
TRACE(OTHER, logout("CU list empty\n"));
|
||||
/* List is empty. Now CU is idle or suspended. */
|
||||
+ s->busy = false;
|
||||
}
|
||||
|
||||
static void eepro100_cu_command(EEPRO100State * s, uint8_t val)
|
||||
--
|
||||
2.27.0
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user