linux-kernel - Re: [PATCH] fs: replace int param with size_t for seq_open

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]

Message-ID: <5404715F.9070707@codethink.co.uk>
Date:	Mon, 01 Sep 2014 14:15:11 +0100
From:	Rob Jones <rob.jones@...ethink.co.uk>
To:	linux-kernel@...r.kernel.org
CC:	linux-fsdevel@...r.kernel.org, viro@...iv.org.uk,
	akpm@...ux-foundation.org, linux-kernel@...ethink.co.uk
Subject: Re: [PATCH] fs: replace int param with size_t for seq_open_private()

Resubmitting - mistranscribed Alexander Viro's email address.

On 01/09/14 14:13, Rob Jones wrote:
> also for __seq_open_private()
>
> Signed-off-by: Rob Jones <rob.jones@...ethink.co.uk>
> ---
>   fs/seq_file.c            |    4 ++--
>   include/linux/seq_file.h |    4 ++--
>   2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/fs/seq_file.c b/fs/seq_file.c
> index 1d641bb..dc2dfec 100644
> --- a/fs/seq_file.c
> +++ b/fs/seq_file.c
> @@ -640,7 +640,7 @@ int seq_release_private(struct inode *inode, struct file *file)
>   EXPORT_SYMBOL(seq_release_private);
>
>   void *__seq_open_private(struct file *f, const struct seq_operations *ops,
> -		int psize)
> +		size_t psize)
>   {
>   	int rc;
>   	void *private;
> @@ -666,7 +666,7 @@ out:
>   EXPORT_SYMBOL(__seq_open_private);
>
>   int seq_open_private(struct file *filp, const struct seq_operations *ops,
> -		int psize)
> +		size_t psize)
>   {
>   	return __seq_open_private(filp, ops, psize) ? 0 : -ENOMEM;
>   }
> diff --git a/include/linux/seq_file.h b/include/linux/seq_file.h
> index 52e0097..9382339 100644
> --- a/include/linux/seq_file.h
> +++ b/include/linux/seq_file.h
> @@ -140,8 +140,8 @@ static inline int seq_nodemask_list(struct seq_file *m, nodemask_t *mask)
>   int single_open(struct file *, int (*)(struct seq_file *, void *), void *);
>   int single_open_size(struct file *, int (*)(struct seq_file *, void *), void *, size_t);
>   int single_release(struct inode *, struct file *);
> -void *__seq_open_private(struct file *, const struct seq_operations *, int);
> -int seq_open_private(struct file *, const struct seq_operations *, int);
> +void *__seq_open_private(struct file *, const struct seq_operations *, size_t);
> +int seq_open_private(struct file *, const struct seq_operations *, size_t);
>   int seq_release_private(struct inode *, struct file *);
>   int seq_put_decimal_ull(struct seq_file *m, char delimiter,
>   			unsigned long long num);
>

-- 
Rob Jones
Codethink Ltd
mailto:rob.jones@...ethink.co.uk
tel:+44 161 236 5575
--
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/