Memory allocated with trigger_data_alloc() require trigger_data_free()
for proper cleanup.

Replace kfree() with trigger_data_free() to fix this.

Found via static analysis and code review.

Fixes: e1f187d09e11 ("tracing: Have existing event_command.parse() 
implementations use helpers")
Cc: [email protected]
Signed-off-by: Miaoqian Lin <[email protected]>
---
 kernel/trace/trace_events_hist.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/trace/trace_events_hist.c b/kernel/trace/trace_events_hist.c
index 5e6e70540eef..f9886fff7123 100644
--- a/kernel/trace/trace_events_hist.c
+++ b/kernel/trace/trace_events_hist.c
@@ -6902,7 +6902,7 @@ static int event_hist_trigger_parse(struct event_command 
*cmd_ops,
 
        remove_hist_vars(hist_data);
 
-       kfree(trigger_data);
+       trigger_data_free(trigger_data);
 
        destroy_hist_data(hist_data);
        goto out;
-- 
2.25.1


Reply via email to