The initialization of ret to 0 was redundant since ret is set to meaningful 
values later in the function. This change simplifies the code and avoids 
confusion about its initial state.

The issue was reported by Coverity Scan.

Report:
CID 1633338 Unused value - An assigned value that is never used may represent 
unnecessary computation, an incorrect algorithm, or possibly the need for 
cleanup or refactoring.
In register_ftrace_graph: A value assigned to a variable is never used. 
(CWE-563)


Signed-off-by: MottiKumar Babu <mottikumarb...@gmail.com>
---
 kernel/trace/fgraph.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/trace/fgraph.c b/kernel/trace/fgraph.c
index 69e226a48daa..b13ab2ad0e88 100644
--- a/kernel/trace/fgraph.c
+++ b/kernel/trace/fgraph.c
@@ -1249,7 +1249,7 @@ int register_ftrace_graph(struct fgraph_ops *gops)
 {
        static bool fgraph_initialized;
        int command = 0;
-       int ret = 0;
+       int ret;
        int i = -1;
 
        guard(mutex)(&ftrace_lock);
-- 
2.43.0


Reply via email to