Smatch complains that "ret" could be uninitialized if we don't enter the loop. I don't know if that's possible, but it's nicer to return a literal zero instead. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> --- kernel/trace/trace_syscalls.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/trace/trace_syscalls.c b/kernel/trace/trace_syscalls.c index 73140d80dd46..63528f458826 100644 --- a/kernel/trace/trace_syscalls.c +++ b/kernel/trace/trace_syscalls.c @@ -286,7 +286,7 @@ static int __init syscall_enter_define_fields(struct trace_event_call *call) offset += sizeof(unsigned long); } - return ret; + return 0; } static void ftrace_syscall_enter(void *data, struct pt_regs *regs, long id) -- 2.11.0
