[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 11/11] hw/net/cadence_gem: enforce 32 bits variable size for CRC
|
From: |
Luc Michel |
|
Subject: |
[PATCH 11/11] hw/net/cadence_gem: enforce 32 bits variable size for CRC |
|
Date: |
Tue, 17 Oct 2023 21:44:22 +0200 |
The CRC was stored in an unsigned variable in gem_receive. Change it for
a uint32_t to ensure we have the correct variable size here.
Signed-off-by: Luc Michel <luc.michel@amd.com>
---
hw/net/cadence_gem.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/net/cadence_gem.c b/hw/net/cadence_gem.c
index 21146f4242..d52530bae4 100644
--- a/hw/net/cadence_gem.c
+++ b/hw/net/cadence_gem.c
@@ -1103,11 +1103,11 @@ static ssize_t gem_receive(NetClientState *nc, const
uint8_t *buf, size_t size)
/* Strip of FCS field ? (usually yes) */
if (FIELD_EX32(s->regs[R_NWCFG], NWCFG, FCS_REMOVE)) {
rxbuf_ptr = (void *)buf;
} else {
- unsigned crc_val;
+ uint32_t crc_val;
if (size > MAX_FRAME_SIZE - sizeof(crc_val)) {
size = MAX_FRAME_SIZE - sizeof(crc_val);
}
bytes_to_copy = size;
--
2.39.2
- [PATCH 02/11] hw/net/cadence_gem: use FIELD for screening registers, (continued)
- [PATCH 02/11] hw/net/cadence_gem: use FIELD for screening registers, Luc Michel, 2023/10/17
- [PATCH 04/11] hw/net/cadence_gem: use FIELD to describe NWCFG register fields, Luc Michel, 2023/10/17
- [PATCH 10/11] hw/net/cadence_gem: perform PHY access on write only, Luc Michel, 2023/10/17
- [PATCH 08/11] hw/net/cadence_gem: use FIELD to describe DESCONF6 register fields, Luc Michel, 2023/10/17
- [PATCH 09/11] hw/net/cadence_gem: use FIELD to describe PHYMNTNC register fields, Luc Michel, 2023/10/17
- [PATCH 11/11] hw/net/cadence_gem: enforce 32 bits variable size for CRC,
Luc Michel <=
- [PATCH 03/11] hw/net/cadence_gem: use FIELD to describe NWCTRL register fields, Luc Michel, 2023/10/17
- [PATCH 05/11] hw/net/cadence_gem: use FIELD to describe DMACFG register fields, Luc Michel, 2023/10/17
- Re: [PATCH 00/11] Various updates for the Cadence GEM model, Peter Maydell, 2023/10/27