rtla: Fix segfault on multiple SIGINTs

Detach stop_trace() from SIGINT/SIGALRM on tool clean-up to prevent it
from crashing RTLA by accessing freed memory.

This prevents a crash when multiple SIGINTs are received.

Fixes: d6899e560366 ("rtla/timerlat_hist: Abort event processing on second signal")
Fixes: 80967b354a76 ("rtla/timerlat_top: Abort event processing on second signal")
Reviewed-by: Wander Lairson Costa <wander@redhat.com>
Link: https://lore.kernel.org/r/20260310160725.144443-1-tglozar@redhat.com
Signed-off-by: Tomas Glozar <tglozar@redhat.com>

diff --git a/tools/tracing/rtla/src/common.c b/tools/tracing/rtla/src/common.c
index 50d8aafdfd3b911bbcd03ab42fa2ab3cb3b723ce..35e3d3aa922e2d1dceb7931036b9f0b90df20931 100644 (file)
--- a/tools/tracing/rtla/src/common.c
+++ b/tools/tracing/rtla/src/common.c
@@ -41,6 +41,18 @@ static void set_signals(struct common_params *params)
        }
 }
 
+/*
+ * unset_signals - unsets the signals to stop the tool
+ */
+static void unset_signals(struct common_params *params)
+{
+       signal(SIGINT, SIG_DFL);
+       if (params->duration) {
+               alarm(0);
+               signal(SIGALRM, SIG_DFL);
+       }
+}
+
 /*
  * getopt_auto - auto-generates optstring from long_options
  */
@@ -351,7 +363,7 @@ int run_tool(struct tool_ops *ops, int argc, char *argv[])
 
        retval = ops->main(tool);
        if (retval)
-               goto out_trace;
+               goto out_signals;
 
        if (params->user_workload && !params->user.stopped_running) {
                params->user.should_run = 0;
@@ -373,6 +385,8 @@ int run_tool(struct tool_ops *ops, int argc, char *argv[])
        if (ops->analyze)
                ops->analyze(tool, stopped);
 
+out_signals:
+       unset_signals(params);
 out_trace:
        trace_events_destroy(&tool->record->trace, params->events);
        params->events = NULL;