* [PATCH] gpio: fix line flag validation in lineevent_create
@ 2019-09-09 3:24 Kent Gibson
2019-09-09 7:31 ` Bartosz Golaszewski
0 siblings, 1 reply; 3+ messages in thread
From: Kent Gibson @ 2019-09-09 3:24 UTC (permalink / raw)
To: linux-gpio; +Cc: Kent Gibson
lineevent_create should not allow any of GPIOHANDLE_REQUEST_OUTPUT,
GPIOHANDLE_REQUEST_OPEN_DRAIN or GPIOHANDLE_REQUEST_OPEN_SOURCE to be set.
Signed-off-by: Kent Gibson <warthog618@gmail.com>
---
drivers/gpio/gpiolib.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
index cca749010cd0..5499ec7bc783 100644
--- a/drivers/gpio/gpiolib.c
+++ b/drivers/gpio/gpiolib.c
@@ -918,7 +918,7 @@ static int lineevent_create(struct gpio_device *gdev, void __user *ip)
goto out_free_label;
}
- /* Return an error if a unknown flag is set */
+ /* Return an error if an unknown flag is set */
if ((lflags & ~GPIOHANDLE_REQUEST_VALID_FLAGS) ||
(eflags & ~GPIOEVENT_REQUEST_VALID_FLAGS)) {
ret = -EINVAL;
@@ -926,7 +926,9 @@ static int lineevent_create(struct gpio_device *gdev, void __user *ip)
}
/* This is just wrong: we don't look for events on output lines */
- if (lflags & GPIOHANDLE_REQUEST_OUTPUT) {
+ if ((lflags & GPIOHANDLE_REQUEST_OUTPUT) ||
+ (lflags & GPIOHANDLE_REQUEST_OPEN_DRAIN) ||
+ (lflags & GPIOHANDLE_REQUEST_OPEN_SOURCE)) {
ret = -EINVAL;
goto out_free_label;
}
@@ -940,10 +942,6 @@ static int lineevent_create(struct gpio_device *gdev, void __user *ip)
if (lflags & GPIOHANDLE_REQUEST_ACTIVE_LOW)
set_bit(FLAG_ACTIVE_LOW, &desc->flags);
- if (lflags & GPIOHANDLE_REQUEST_OPEN_DRAIN)
- set_bit(FLAG_OPEN_DRAIN, &desc->flags);
- if (lflags & GPIOHANDLE_REQUEST_OPEN_SOURCE)
- set_bit(FLAG_OPEN_SOURCE, &desc->flags);
ret = gpiod_direction_input(desc);
if (ret)
--
2.23.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] gpio: fix line flag validation in lineevent_create
2019-09-09 3:24 [PATCH] gpio: fix line flag validation in lineevent_create Kent Gibson
@ 2019-09-09 7:31 ` Bartosz Golaszewski
2019-09-09 8:07 ` Bartosz Golaszewski
0 siblings, 1 reply; 3+ messages in thread
From: Bartosz Golaszewski @ 2019-09-09 7:31 UTC (permalink / raw)
To: Kent Gibson; +Cc: open list:GPIO SUBSYSTEM
pon., 9 wrz 2019 o 05:24 Kent Gibson <warthog618@gmail.com> napisał(a):
>
Hi Kent,
thanks for spotting this. Just single nit below:
> lineevent_create should not allow any of GPIOHANDLE_REQUEST_OUTPUT,
> GPIOHANDLE_REQUEST_OPEN_DRAIN or GPIOHANDLE_REQUEST_OPEN_SOURCE to be set.
>
> Signed-off-by: Kent Gibson <warthog618@gmail.com>
> ---
> drivers/gpio/gpiolib.c | 10 ++++------
> 1 file changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
> index cca749010cd0..5499ec7bc783 100644
> --- a/drivers/gpio/gpiolib.c
> +++ b/drivers/gpio/gpiolib.c
> @@ -918,7 +918,7 @@ static int lineevent_create(struct gpio_device *gdev, void __user *ip)
> goto out_free_label;
> }
>
> - /* Return an error if a unknown flag is set */
> + /* Return an error if an unknown flag is set */
Please don't sneak in changes unrelated to the patch. If you want to
fix the typo - do it in a separate one.
> if ((lflags & ~GPIOHANDLE_REQUEST_VALID_FLAGS) ||
> (eflags & ~GPIOEVENT_REQUEST_VALID_FLAGS)) {
> ret = -EINVAL;
> @@ -926,7 +926,9 @@ static int lineevent_create(struct gpio_device *gdev, void __user *ip)
> }
>
> /* This is just wrong: we don't look for events on output lines */
> - if (lflags & GPIOHANDLE_REQUEST_OUTPUT) {
> + if ((lflags & GPIOHANDLE_REQUEST_OUTPUT) ||
> + (lflags & GPIOHANDLE_REQUEST_OPEN_DRAIN) ||
> + (lflags & GPIOHANDLE_REQUEST_OPEN_SOURCE)) {
> ret = -EINVAL;
> goto out_free_label;
> }
> @@ -940,10 +942,6 @@ static int lineevent_create(struct gpio_device *gdev, void __user *ip)
>
> if (lflags & GPIOHANDLE_REQUEST_ACTIVE_LOW)
> set_bit(FLAG_ACTIVE_LOW, &desc->flags);
> - if (lflags & GPIOHANDLE_REQUEST_OPEN_DRAIN)
> - set_bit(FLAG_OPEN_DRAIN, &desc->flags);
> - if (lflags & GPIOHANDLE_REQUEST_OPEN_SOURCE)
> - set_bit(FLAG_OPEN_SOURCE, &desc->flags);
>
> ret = gpiod_direction_input(desc);
> if (ret)
> --
> 2.23.0
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] gpio: fix line flag validation in lineevent_create
2019-09-09 7:31 ` Bartosz Golaszewski
@ 2019-09-09 8:07 ` Bartosz Golaszewski
0 siblings, 0 replies; 3+ messages in thread
From: Bartosz Golaszewski @ 2019-09-09 8:07 UTC (permalink / raw)
To: Kent Gibson; +Cc: open list:GPIO SUBSYSTEM
pon., 9 wrz 2019 o 09:31 Bartosz Golaszewski <brgl@bgdev.pl> napisał(a):
>
> pon., 9 wrz 2019 o 05:24 Kent Gibson <warthog618@gmail.com> napisał(a):
> >
>
> Hi Kent,
>
> thanks for spotting this. Just single nit below:
>
> > lineevent_create should not allow any of GPIOHANDLE_REQUEST_OUTPUT,
> > GPIOHANDLE_REQUEST_OPEN_DRAIN or GPIOHANDLE_REQUEST_OPEN_SOURCE to be set.
> >
> > Signed-off-by: Kent Gibson <warthog618@gmail.com>
> > ---
> > drivers/gpio/gpiolib.c | 10 ++++------
> > 1 file changed, 4 insertions(+), 6 deletions(-)
> >
> > diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
> > index cca749010cd0..5499ec7bc783 100644
> > --- a/drivers/gpio/gpiolib.c
> > +++ b/drivers/gpio/gpiolib.c
> > @@ -918,7 +918,7 @@ static int lineevent_create(struct gpio_device *gdev, void __user *ip)
> > goto out_free_label;
> > }
> >
> > - /* Return an error if a unknown flag is set */
> > + /* Return an error if an unknown flag is set */
>
> Please don't sneak in changes unrelated to the patch. If you want to
> fix the typo - do it in a separate one.
>
> > if ((lflags & ~GPIOHANDLE_REQUEST_VALID_FLAGS) ||
> > (eflags & ~GPIOEVENT_REQUEST_VALID_FLAGS)) {
> > ret = -EINVAL;
> > @@ -926,7 +926,9 @@ static int lineevent_create(struct gpio_device *gdev, void __user *ip)
> > }
> >
> > /* This is just wrong: we don't look for events on output lines */
> > - if (lflags & GPIOHANDLE_REQUEST_OUTPUT) {
> > + if ((lflags & GPIOHANDLE_REQUEST_OUTPUT) ||
> > + (lflags & GPIOHANDLE_REQUEST_OPEN_DRAIN) ||
> > + (lflags & GPIOHANDLE_REQUEST_OPEN_SOURCE)) {
> > ret = -EINVAL;
> > goto out_free_label;
> > }
> > @@ -940,10 +942,6 @@ static int lineevent_create(struct gpio_device *gdev, void __user *ip)
> >
> > if (lflags & GPIOHANDLE_REQUEST_ACTIVE_LOW)
> > set_bit(FLAG_ACTIVE_LOW, &desc->flags);
> > - if (lflags & GPIOHANDLE_REQUEST_OPEN_DRAIN)
> > - set_bit(FLAG_OPEN_DRAIN, &desc->flags);
> > - if (lflags & GPIOHANDLE_REQUEST_OPEN_SOURCE)
> > - set_bit(FLAG_OPEN_SOURCE, &desc->flags);
> >
> > ret = gpiod_direction_input(desc);
> > if (ret)
> > --
> > 2.23.0
> >
Ok, so seeing that we'll still have another week for fixes, I thought
it would be nice to get those in before v5.3. I eventually applied
both patches and simply removed the typo fix.
Bart
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-09-09 8:08 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-09 3:24 [PATCH] gpio: fix line flag validation in lineevent_create Kent Gibson
2019-09-09 7:31 ` Bartosz Golaszewski
2019-09-09 8:07 ` Bartosz Golaszewski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).