mirror of
https://github.com/torvalds/linux.git
synced 2024-11-20 02:51:44 +00:00
tty: double unlock on error in ptmx_open()
The problem here is that we called mutex_unlock(&devpts_mutex) on the error path when we weren't holding the lock. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
157a4b311c
commit
79d7532092
@ -618,13 +618,12 @@ static int ptmx_open(struct inode *inode, struct file *filp)
|
||||
/* find a device that is not in use. */
|
||||
mutex_lock(&devpts_mutex);
|
||||
index = devpts_new_index(inode);
|
||||
mutex_unlock(&devpts_mutex);
|
||||
if (index < 0) {
|
||||
retval = index;
|
||||
goto err_file;
|
||||
}
|
||||
|
||||
mutex_unlock(&devpts_mutex);
|
||||
|
||||
mutex_lock(&tty_mutex);
|
||||
tty = tty_init_dev(ptm_driver, index);
|
||||
|
||||
@ -659,7 +658,6 @@ out:
|
||||
mutex_unlock(&tty_mutex);
|
||||
devpts_kill_index(inode, index);
|
||||
err_file:
|
||||
mutex_unlock(&devpts_mutex);
|
||||
tty_free_file(filp);
|
||||
return retval;
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user