fs/devpts/inode.c: correctly check d_alloc_name() return code in devpts_pty_new()
authorAndrey Vagin <avagin@openvz.org>
Tue, 22 Mar 2011 23:35:11 +0000 (16:35 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 23 Mar 2011 00:44:17 +0000 (17:44 -0700)
d_alloc_name return NULL in case error, but we expect errno in
devpts_pty_new.

Addresses http://bugzilla.openvz.org/show_bug.cgi?id=1758

Signed-off-by: Andrey Vagin <avagin@openvz.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/devpts/inode.c

index c6bd815..2f27e57 100644 (file)
@@ -502,7 +502,7 @@ int devpts_pty_new(struct inode *ptmx_inode, struct tty_struct *tty)
        mutex_lock(&root->d_inode->i_mutex);
 
        dentry = d_alloc_name(root, s);
-       if (!IS_ERR(dentry)) {
+       if (dentry) {
                d_add(dentry, inode);
                fsnotify_create(root->d_inode, dentry);
        } else {