Messages in this thread Patch in this message | | | From | cgel.zte@gmail ... | Subject | [PATCH linux-next] tracing: Remove the unneeded result variable | Date | Fri, 26 Aug 2022 07:27:36 +0000 |
| |
From: ye xingchen <ye.xingchen@zte.com.cn>
Return the value simple_read_from_buffer() directly instead of storing it in another redundant variable.
Reported-by: Zeal Robot <zealci@zte.com.cn> Signed-off-by: ye xingchen <ye.xingchen@zte.com.cn> --- kernel/trace/trace_events.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index 0356cae0cf74..9e4dd1720fff 100644 --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c @@ -1435,7 +1435,6 @@ system_enable_read(struct file *filp, char __user *ubuf, size_t cnt, struct trace_array *tr = dir->tr; char buf[2]; int set = 0; - int ret; mutex_lock(&event_mutex); list_for_each_entry(file, &tr->events, list) { @@ -1465,9 +1464,7 @@ system_enable_read(struct file *filp, char __user *ubuf, size_t cnt, buf[0] = set_to_char[set]; buf[1] = '\n'; - ret = simple_read_from_buffer(ubuf, cnt, ppos, buf, 2); - - return ret; + return simple_read_from_buffer(ubuf, cnt, ppos, buf, 2); } static ssize_t -- 2.25.1
| |