diff options
author | Prasad J Pandit <pjp@fedoraproject.org> | 2016-10-12 14:40:55 +0530 |
---|---|---|
committer | Jason Wang <jasowang@redhat.com> | 2016-10-26 09:57:59 +0800 |
commit | 8caed3d564672e8bc6d2e4c6a35228afd01f4723 (patch) | |
tree | 5e8887f57d22e9e07b0982e018d51b6380f977a6 | |
parent | 2634ab7fe29b3f75d0865b719caf8f310d634aae (diff) | |
download | qemu-8caed3d564672e8bc6d2e4c6a35228afd01f4723.zip |
net: rocker: set limit to DMA buffer size
Rocker network switch emulator has test registers to help debug
DMA operations. While testing host DMA access, a buffer address
is written to register 'TEST_DMA_ADDR' and its size is written to
register 'TEST_DMA_SIZE'. When performing TEST_DMA_CTRL_INVERT
test, if DMA buffer size was greater than 'INT_MAX', it leads to
an invalid buffer access. Limit the DMA buffer size to avoid it.
Reported-by: Huawei PSIRT <psirt@huawei.com>
Signed-off-by: Prasad J Pandit <pjp@fedoraproject.org>
Reviewed-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
-rw-r--r-- | hw/net/rocker/rocker.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/net/rocker/rocker.c b/hw/net/rocker/rocker.c index 30f2ce417b..e9d215aa4d 100644 --- a/hw/net/rocker/rocker.c +++ b/hw/net/rocker/rocker.c @@ -860,7 +860,7 @@ static void rocker_io_writel(void *opaque, hwaddr addr, uint32_t val) rocker_msix_irq(r, val); break; case ROCKER_TEST_DMA_SIZE: - r->test_dma_size = val; + r->test_dma_size = val & 0xFFFF; break; case ROCKER_TEST_DMA_ADDR + 4: r->test_dma_addr = ((uint64_t)val) << 32 | r->lower32; |