[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1] arm: check bit index before usage
From: |
P J P |
Subject: |
Re: [Qemu-devel] [PATCH v1] arm: check bit index before usage |
Date: |
Fri, 26 Oct 2018 15:07:34 +0530 (IST) |
+-- On Fri, 26 Oct 2018, Peter Maydell wrote --+
| > ===
| > diff --git a/hw/arm/strongarm.c b/hw/arm/strongarm.c
| > index ec2627374d..dd8c4b1f2e 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 & 0x3fffff;
| > strongarm_gpio_handler_update(s);
| > break;
| >
| > case GPSR: /* GPIO Pin-Output Set registers */
| > - s->olevel |= value;
| > + s->olevel |= value & 0x3fffff;
| > strongarm_gpio_handler_update(s);
| > break;
| > ===
| >
| > does this seem okay?
|
| Yes, that's what I had in mind.
Cool, patch sent.
Thank you.
--
Prasad J Pandit / Red Hat Product Security Team
47AF CE69 3A90 54AA 9045 1053 DD13 3D32 FE5B 041F