[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 2/3] hw/net/stellaris_enet: Avoid unintended sign ex
From: |
Peter Maydell |
Subject: |
[Qemu-devel] [PATCH 2/3] hw/net/stellaris_enet: Avoid unintended sign extension |
Date: |
Mon, 17 Feb 2014 14:37:33 +0000 |
Add a cast to avoid an unintended sign extension that
would mean we returned 0xffffffff in the high 32 bits
for an IA0 read if bit 31 in the MAC address was 1.
(This is harmless since we'll only be doing 4 byte
reads, but it could be confusing, so best avoided.)
Signed-off-by: Peter Maydell <address@hidden>
---
hw/net/stellaris_enet.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw/net/stellaris_enet.c b/hw/net/stellaris_enet.c
index 9dd77f7..d04e6a4 100644
--- a/hw/net/stellaris_enet.c
+++ b/hw/net/stellaris_enet.c
@@ -176,7 +176,8 @@ static uint64_t stellaris_enet_read(void *opaque, hwaddr
offset,
return val;
case 0x14: /* IA0 */
return s->conf.macaddr.a[0] | (s->conf.macaddr.a[1] << 8)
- | (s->conf.macaddr.a[2] << 16) | (s->conf.macaddr.a[3] << 24);
+ | (s->conf.macaddr.a[2] << 16)
+ | ((uint32_t)s->conf.macaddr.a[3] << 24);
case 0x18: /* IA1 */
return s->conf.macaddr.a[4] | (s->conf.macaddr.a[5] << 8);
case 0x1c: /* THR */
--
1.8.5
- [Qemu-devel] [PATCH 0/3] ARM: three easy patches for coverity-reported issues, Peter Maydell, 2014/02/17
- [Qemu-devel] [PATCH 2/3] hw/net/stellaris_enet: Avoid unintended sign extension,
Peter Maydell <=
- [Qemu-devel] [PATCH 3/3] hw/timer/arm_timer: Avoid array overrun for bad addresses, Peter Maydell, 2014/02/17
- [Qemu-devel] [PATCH 1/3] hw/misc/arm_sysctl: Fix bad boundary check on mb clock accesses, Peter Maydell, 2014/02/17
- Re: [Qemu-devel] [PATCH 0/3] ARM: three easy patches for coverity-reported issues, Paolo Bonzini, 2014/02/17
- Re: [Qemu-devel] [PATCH 0/3] ARM: three easy patches for coverity-reported issues, Andreas Färber, 2014/02/17
- Re: [Qemu-devel] [PATCH 0/3] ARM: three easy patches for coverity-reported issues, Peter Maydell, 2014/02/18
- Re: [Qemu-devel] [PATCH 0/3] ARM: three easy patches for coverity-reported issues, Andreas Färber, 2014/02/18
- Re: [Qemu-devel] [PATCH 0/3] ARM: three easy patches for coverity-reported issues, Peter Maydell, 2014/02/18
- Re: [Qemu-devel] [PATCH 0/3] ARM: three easy patches for coverity-reported issues, Paolo Bonzini, 2014/02/18
- Re: [Qemu-devel] [PATCH 0/3] ARM: three easy patches for coverity-reported issues, Peter Maydell, 2014/02/18
- Re: [Qemu-devel] [PATCH 0/3] ARM: three easy patches for coverity-reported issues, Paolo Bonzini, 2014/02/18