diff options
| author | Alexey Dobriyan <adobriyan@gmail.com> | 2008-08-02 07:30:48 +0400 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2008-08-25 01:18:03 -0400 |
| commit | cc996099174dc05b35b7a29301026987990e7f8c (patch) | |
| tree | bfdb337167e8d33d5fd88a17503821f189485d3e | |
| parent | 6450f65168bcf3c03b5fb44c2fe96682c0d3086b (diff) | |
| download | kernel_samsung_smdk4412-cc996099174dc05b35b7a29301026987990e7f8c.zip kernel_samsung_smdk4412-cc996099174dc05b35b7a29301026987990e7f8c.tar.gz kernel_samsung_smdk4412-cc996099174dc05b35b7a29301026987990e7f8c.tar.bz2 | |
[PATCH] proc: inode number fixlet
Ouch, if number taken from IDA is too big, the intent was to signal an
error, not check for overflow and still do overflowing addition.
One still needs 2^28 proc entries to notice this.
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
| -rw-r--r-- | fs/proc/generic.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/proc/generic.c b/fs/proc/generic.c index 4fb81e9..bca0f81 100644 --- a/fs/proc/generic.c +++ b/fs/proc/generic.c @@ -330,6 +330,7 @@ retry: spin_lock(&proc_inum_lock); ida_remove(&proc_inum_ida, i); spin_unlock(&proc_inum_lock); + return 0; } return PROC_DYNAMIC_FIRST + i; } |