diff options
author | yan <clouds.yan@gmail.com> | 2012-10-04 20:15:38 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-10-05 14:05:17 -0400 |
commit | 620727506dc6da0562fa4f6950dedb8a51bd8237 (patch) | |
tree | 138a5b5f5ef6376c6a593bc99bcda8d93fb34622 /fs/proc/generic.c | |
parent | 2aa362c49c314a98fb9aebbd7760a461667bac05 (diff) |
proc: return -ENOMEM when inode allocation failed
If proc_get_inode() returns NULL then presumably it encountered memory
exhaustion. proc_lookup_de() should return -ENOMEM in this case, not
-EINVAL.
Signed-off-by: yan <clouds.yan@gmail.com>
Cc: Ryan Mallon <rmallon@gmail.com>
Cc: Cong Wang <xiyou.wangcong@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/proc/generic.c')
-rw-r--r-- | fs/proc/generic.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/proc/generic.c b/fs/proc/generic.c index b3647fe6a608..9e8f63164309 100644 --- a/fs/proc/generic.c +++ b/fs/proc/generic.c | |||
@@ -427,7 +427,7 @@ struct dentry *proc_lookup_de(struct proc_dir_entry *de, struct inode *dir, | |||
427 | if (!memcmp(dentry->d_name.name, de->name, de->namelen)) { | 427 | if (!memcmp(dentry->d_name.name, de->name, de->namelen)) { |
428 | pde_get(de); | 428 | pde_get(de); |
429 | spin_unlock(&proc_subdir_lock); | 429 | spin_unlock(&proc_subdir_lock); |
430 | error = -EINVAL; | 430 | error = -ENOMEM; |
431 | inode = proc_get_inode(dir->i_sb, de); | 431 | inode = proc_get_inode(dir->i_sb, de); |
432 | goto out_unlock; | 432 | goto out_unlock; |
433 | } | 433 | } |