linux-kernel - Re: [PATCH] iio: event: Remove negative error code from iio_event

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Sat, 8 Aug 2015 17:20:38 +0100
From:	Jonathan Cameron <jic23@...nel.org>
To:	Cristina Opriceana <cristina.opriceana@...il.com>
Cc:	knaack.h@....de, lars@...afoo.de, pmeerw@...erw.net,
	linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org,
	daniel.baluta@...el.com
Subject: Re: [PATCH] iio: event: Remove negative error code from
 iio_event_poll

On 03/08/15 11:00, Cristina Opriceana wrote:
> Negative return values are not supported by iio_event_poll since
> its return type is unsigned int.
> 
> Signed-off-by: Cristina Opriceana <cristina.opriceana@...il.com>
As with the other one, the docs aren't yet there in the fixes tree where
I have applied this (by hand).  I've also added the fixes line and
marked it for stable.

Could you resend the docs updates after that patch has worked it's
way through to the togreg branch of iio.git

(I'll try and remember this as well!)

Thanks,

Jonathan
> ---
>  drivers/iio/industrialio-event.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/iio/industrialio-event.c b/drivers/iio/industrialio-event.c
> index b2f63f9..eb44dab 100644
> --- a/drivers/iio/industrialio-event.c
> +++ b/drivers/iio/industrialio-event.c
> @@ -80,7 +80,7 @@ EXPORT_SYMBOL(iio_push_event);
>   * @wait:	Poll table pointer to add the wait queue on
>   *
>   * Return: (POLLIN | POLLRDNORM) if data is available for reading
> - *	   or a negative error code on failure
> + *	   or 0 on failure
>   */
>  static unsigned int iio_event_poll(struct file *filep,
>  			     struct poll_table_struct *wait)
> @@ -90,7 +90,7 @@ static unsigned int iio_event_poll(struct file *filep,
>  	unsigned int events = 0;
>  
>  	if (!indio_dev->info)
> -		return -ENODEV;
> +		return events;
>  
>  	poll_wait(filep, &ev_int->wait, wait);
>  
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/