]> err.no Git - linux-2.6/commit
[PATCH] Fix double decrement of mqueue_mnt->mnt_count in sys_mq_open
authorAlexander Viro <aviro@redhat.com>
Sat, 14 Jan 2006 20:29:55 +0000 (15:29 -0500)
committerLinus Torvalds <torvalds@g5.osdl.org>
Sat, 14 Jan 2006 20:38:17 +0000 (12:38 -0800)
commit7c7dce9209161eb260cdf9e9172f72c3a02379e6
tree81135190b6aadfc9126dcbff63710cd97fe5b6a1
parent12dbf3fc4d06d2c0c4c44dc0612df04248b3cfd3
[PATCH] Fix double decrement of mqueue_mnt->mnt_count in sys_mq_open

Fixed the refcounting on failure exits in sys_mq_open() and
cleaned the logics up.  Rules are actually pretty simple - dentry_open()
expects vfsmount and dentry to be pinned down and it either transfers
them into created struct file or drops them.  Old code had been very
confused in that area - if dentry_open() had failed either in do_open()
or do_create(), we ended up dentry and mqueue_mnt dropped twice, once
by dentry_open() cleanup and then by sys_mq_open().

Fix consists of making the rules for do_create() and do_open()
same as for dentry_open() and updating the sys_mq_open() accordingly;
that actually leads to more straightforward code and less work on
normal path.

Signed-off-by: Al Viro <aviro@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
ipc/mqueue.c