Re: [PATCH v2 04/16] perf python: Fix memory leak in pyrf_evlist__get_pollfd

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:

Pre-existing issues:
- [Low] Missing explicit POSIX libc header inclusions for `<stdlib.h>` and `<stdio.h>`.
--

commit b37f20df0f07ef7454731749a918a55a91b0c7db
Author: Ian Rogers <irogers@google.com>

perf python: Fix memory leak in pyrf_evlist__get_pollfd

This commit fixes a Python list object leak in pyrf_evlist__get_pollfd() by
adding a missing Py_DECREF on the error exit path.

> diff --git a/tools/perf/util/python.c b/tools/perf/util/python.c
> index 11e1f39c1bea9..550f1e7d14499 100644
> --- a/tools/perf/util/python.c
> +++ b/tools/perf/util/python.c

[Severity: Low]
This isn't a bug introduced by this patch, but there is a pre-existing issue
where the top-level includes section is missing explicit POSIX libc headers:

#include <inttypes.h>
#include <string.h>

#include <linux/err.h>

The file relies on POSIX standard functions like calloc() and free() which
require <stdlib.h>, as well as asprintf() which requires <stdio.h>. Could
these be explicitly added to ensure musl compilation compatibility?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260624051535.3584977-1-irogers@google.com?part=4