diff options
| author | Samasth Norway Ananda <[email protected]> | 2024-05-09 17:34:24 -0700 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <[email protected]> | 2024-05-10 11:28:11 -0300 |
| commit | 09541603462c399c7408d50295db99b4b8042eaa (patch) | |
| tree | f687f6ec3791e71bd553c2d132e7cdf106484a8f /tools/perf/scripts/python/export-to-postgresql.py | |
| parent | 230a7a71f92212e723fa435d4ca5922de33ec88a (diff) | |
perf daemon: Fix file leak in daemon_session__control
The open() function returns -1 on error.
The 'control' and 'ack' file descriptors are both initialized with
open() and further validated with 'if' statement.
'if (!control)' would evaluate to 'true' if returned value on error were
'0' but it is actually '-1'.
Fixes: edcaa47958c7438b ("perf daemon: Add 'ping' command")
Signed-off-by: Samasth Norway Ananda <[email protected]>
Cc: Adrian Hunter <[email protected]>
Cc: Alexander Shishkin <[email protected]>
Cc: Ian Rogers <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Mark Rutland <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/export-to-postgresql.py')
0 files changed, 0 insertions, 0 deletions