diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2008-12-10 06:58:59 -0500 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2009-01-04 15:14:40 -0500 |
| commit | 7392906ea915b9a2c14dea32b3604b4e178f82f7 (patch) | |
| tree | 1e4fbe56e3738fade213ef805ec274ea74ac6a1b /ipc | |
| parent | e816f370cbadd2afea9f1a42f232d0636137d563 (diff) | |
| download | lwn-7392906ea915b9a2c14dea32b3604b4e178f82f7.tar.gz lwn-7392906ea915b9a2c14dea32b3604b4e178f82f7.zip | |
sanitize audit_mq_getsetattr()
* get rid of allocations
* make it return void
* don't duplicate parts of audit_dummy_context()
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'ipc')
| -rw-r--r-- | ipc/mqueue.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/ipc/mqueue.c b/ipc/mqueue.c index d9393f8e4c3e..7563611c6615 100644 --- a/ipc/mqueue.c +++ b/ipc/mqueue.c @@ -1150,11 +1150,7 @@ asmlinkage long sys_mq_getsetattr(mqd_t mqdes, omqstat = info->attr; omqstat.mq_flags = filp->f_flags & O_NONBLOCK; if (u_mqstat) { - ret = audit_mq_getsetattr(mqdes, &mqstat); - if (ret != 0) { - spin_unlock(&info->lock); - goto out_fput; - } + audit_mq_getsetattr(mqdes, &mqstat); if (mqstat.mq_flags & O_NONBLOCK) filp->f_flags |= O_NONBLOCK; else |