diff options
author | Alan Cox <alan@redhat.com> | 2008-10-13 05:42:29 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-10-13 12:51:43 -0400 |
commit | 73ec06fc5f5c8e1097a7a4a4ab2d7c6c3a007e81 (patch) | |
tree | 2c244c8b0d853f82e479eaf72ca64d6c9209c6a6 /drivers/char/pty.c | |
parent | 8b0a88d5912ab549d5adac2c8498ecdaae5319a5 (diff) |
tty: Finish fixing up the init_dev interface to use ERR_PTR
Original suggestion and proposal from Sukadev Bhattiprolu.
Signed-off-by: Alan Cox <alan@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/char/pty.c')
-rw-r--r-- | drivers/char/pty.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/char/pty.c b/drivers/char/pty.c index 4e6490bda751..c98450023030 100644 --- a/drivers/char/pty.c +++ b/drivers/char/pty.c | |||
@@ -501,11 +501,13 @@ static int __ptmx_open(struct inode *inode, struct file *filp) | |||
501 | return index; | 501 | return index; |
502 | 502 | ||
503 | mutex_lock(&tty_mutex); | 503 | mutex_lock(&tty_mutex); |
504 | retval = tty_init_dev(ptm_driver, index, &tty, 1); | 504 | tty = tty_init_dev(ptm_driver, index, 1); |
505 | mutex_unlock(&tty_mutex); | 505 | mutex_unlock(&tty_mutex); |
506 | 506 | ||
507 | if (retval) | 507 | if (IS_ERR(tty)) { |
508 | retval = PTR_ERR(tty); | ||
508 | goto out; | 509 | goto out; |
510 | } | ||
509 | 511 | ||
510 | set_bit(TTY_PTY_LOCK, &tty->flags); /* LOCK THE SLAVE */ | 512 | set_bit(TTY_PTY_LOCK, &tty->flags); /* LOCK THE SLAVE */ |
511 | filp->private_data = tty; | 513 | filp->private_data = tty; |