Skip to content

Commit d8cc1ab

Browse files
Li Zefanrostedt
authored andcommitted
trace_stack: Fix seqfile memory leak
Every time we cat stack_trace, we leak memory allocated by seq_open(). Signed-off-by: Li Zefan <lizf@cn.fujitsu.com> LKML-Reference: <4A67D8E8.3020500@cn.fujitsu.com> Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
1 parent e5d490b commit d8cc1ab

1 file changed

Lines changed: 2 additions & 5 deletions

File tree

kernel/trace/trace_stack.c

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -301,17 +301,14 @@ static const struct seq_operations stack_trace_seq_ops = {
301301

302302
static int stack_trace_open(struct inode *inode, struct file *file)
303303
{
304-
int ret;
305-
306-
ret = seq_open(file, &stack_trace_seq_ops);
307-
308-
return ret;
304+
return seq_open(file, &stack_trace_seq_ops);
309305
}
310306

311307
static const struct file_operations stack_trace_fops = {
312308
.open = stack_trace_open,
313309
.read = seq_read,
314310
.llseek = seq_lseek,
311+
.release = seq_release,
315312
};
316313

317314
int

0 commit comments

Comments
 (0)