[PATCH v1] libperf rc_check: Enable implicitly with sanitizers

Ian Rogers posted 1 patch 2 years, 8 months ago
tools/lib/perf/include/internal/rc_check.h | 8 ++++++++
1 file changed, 8 insertions(+)
[PATCH v1] libperf rc_check: Enable implicitly with sanitizers
Posted by Ian Rogers 2 years, 8 months ago
If using leak sanitizer then implicitly enable reference count
checking.

Signed-off-by: Ian Rogers <irogers@google.com>
---
 tools/lib/perf/include/internal/rc_check.h | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/tools/lib/perf/include/internal/rc_check.h b/tools/lib/perf/include/internal/rc_check.h
index c0626d8beb59..d5d771ccdc7b 100644
--- a/tools/lib/perf/include/internal/rc_check.h
+++ b/tools/lib/perf/include/internal/rc_check.h
@@ -5,6 +5,14 @@
 #include <stdlib.h>
 #include <linux/zalloc.h>
 
+/*
+ * Enable reference count checking implicitly with leak checking, which is
+ * integrated into address sanitizer.
+ */
+#if defined(LEAK_SANITIZER) || defined(ADDRESS_SANITIZER)
+#define REFCNT_CHECKING 1
+#endif
+
 /*
  * Shared reference count checking macros.
  *
-- 
2.40.0.634.g4ca3ef3211-goog
Re: [PATCH v1] libperf rc_check: Enable implicitly with sanitizers
Posted by Arnaldo Carvalho de Melo 2 years, 8 months ago
Em Thu, Apr 20, 2023 at 10:18:12AM -0700, Ian Rogers escreveu:
> If using leak sanitizer then implicitly enable reference count
> checking.

Thanks, applied.

- Arnaldo

 
> Signed-off-by: Ian Rogers <irogers@google.com>
> ---
>  tools/lib/perf/include/internal/rc_check.h | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/tools/lib/perf/include/internal/rc_check.h b/tools/lib/perf/include/internal/rc_check.h
> index c0626d8beb59..d5d771ccdc7b 100644
> --- a/tools/lib/perf/include/internal/rc_check.h
> +++ b/tools/lib/perf/include/internal/rc_check.h
> @@ -5,6 +5,14 @@
>  #include <stdlib.h>
>  #include <linux/zalloc.h>
>  
> +/*
> + * Enable reference count checking implicitly with leak checking, which is
> + * integrated into address sanitizer.
> + */
> +#if defined(LEAK_SANITIZER) || defined(ADDRESS_SANITIZER)
> +#define REFCNT_CHECKING 1
> +#endif
> +
>  /*
>   * Shared reference count checking macros.
>   *
> -- 
> 2.40.0.634.g4ca3ef3211-goog
> 

-- 

- Arnaldo