fix memory leaks in tracing_buffers_splice_read()

commit 1ae2293dd6d2f5c823cf97e60b70d03631cd622f upstream.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Willy Tarreau <w@1wt.eu>
This commit is contained in:
Al Viro 2016-09-17 18:31:46 -04:00 committed by syphyr
parent c079079279
commit 32bfd3d812
1 changed files with 9 additions and 8 deletions

View File

@ -5178,11 +5178,6 @@ tracing_buffers_splice_read(struct file *file, loff_t *ppos,
}
#endif
if (splice_grow_spd(pipe, &spd)) {
ret = -ENOMEM;
goto out;
}
if (*ppos & (PAGE_SIZE - 1)) {
ret = -EINVAL;
goto out;
@ -5196,6 +5191,11 @@ tracing_buffers_splice_read(struct file *file, loff_t *ppos,
len &= PAGE_MASK;
}
if (splice_grow_spd(pipe, &spd)) {
ret = -ENOMEM;
goto out;
}
again:
trace_access_lock(iter->cpu_file);
entries = ring_buffer_entries_cpu(iter->trace_buffer->buffer, iter->cpu_file);
@ -5251,21 +5251,22 @@ tracing_buffers_splice_read(struct file *file, loff_t *ppos,
if (!spd.nr_pages) {
if ((file->f_flags & O_NONBLOCK) || (flags & SPLICE_F_NONBLOCK)) {
ret = -EAGAIN;
goto out;
goto out_shrink;
}
mutex_unlock(&trace_types_lock);
ret = iter->trace->wait_pipe(iter);
mutex_lock(&trace_types_lock);
if (ret)
goto out;
goto out_shrink;
if (signal_pending(current)) {
ret = -EINTR;
goto out;
goto out_shrink;
}
goto again;
}
ret = splice_to_pipe(pipe, &spd);
out_shrink:
splice_shrink_spd(&spd);
out:
mutex_unlock(&trace_types_lock);