diff options
| author | Prasad J Pandit <pjp@fedoraproject.org> | 2018-10-30 15:23:56 +0000 |
|---|---|---|
| committer | Peter Maydell <peter.maydell@linaro.org> | 2018-11-02 14:03:33 +0000 |
| commit | 9a93b2fa0ed8de66a9deb9106ce5ddc2a65d68f0 (patch) | |
| tree | b793434e0f323386c7d7df1e250ac0b575e2e906 | |
| parent | 22461bd2e59380c08683044c580c18b1f3c9ce26 (diff) | |
| download | qemu-9a93b2fa0ed8de66a9deb9106ce5ddc2a65d68f0.zip qemu-9a93b2fa0ed8de66a9deb9106ce5ddc2a65d68f0.tar.gz qemu-9a93b2fa0ed8de66a9deb9106ce5ddc2a65d68f0.tar.bz2 | |
strongarm: mask off high[31:28] bits from dir and state registers
The high[31:28] bits of 'direction' and 'state' registers of
SA-1100/SA-1110 device are reserved. Setting them may lead to
OOB 's->handler[]' array access issue. Mask off [31:28] bits to
avoid it.
Reported-by: Moguofang <moguofang@huawei.com>
Signed-off-by: Prasad J Pandit <pjp@fedoraproject.org>
Message-id: 20181030114635.31232-1-ppandit@redhat.com
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
| -rw-r--r-- | hw/arm/strongarm.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/hw/arm/strongarm.c b/hw/arm/strongarm.c index ec26273..644a9c4 100644 --- a/hw/arm/strongarm.c +++ b/hw/arm/strongarm.c @@ -587,12 +587,12 @@ static void strongarm_gpio_write(void *opaque, hwaddr offset, switch (offset) { case GPDR: /* GPIO Pin-Direction registers */ - s->dir = value; + s->dir = value & 0x0fffffff; strongarm_gpio_handler_update(s); break; case GPSR: /* GPIO Pin-Output Set registers */ - s->olevel |= value; + s->olevel |= value & 0x0fffffff; strongarm_gpio_handler_update(s); break; |