diff options
| author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2026-01-27 02:09:37 -0300 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2026-01-27 02:09:37 -0300 |
| commit | f1321cce848c558fde4c0c6bcd5e53f3cefd3af2 (patch) | |
| tree | a8c35160e381ce1ba70b10c0b3711e9f418eef9f /tools/perf/util | |
| parent | 678ed6b707e4b2db250f255d2f959322896dae65 (diff) | |
| download | lwn-f1321cce848c558fde4c0c6bcd5e53f3cefd3af2.tar.gz lwn-f1321cce848c558fde4c0c6bcd5e53f3cefd3af2.zip | |
perf session: Don't write to memory pointed to a const pointer
Since it is freshly allocated just attribute it to a non-const pointer
and then change it via that pointer.
That way we avoid const-correctness warnings in recent glibc versions.
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/util')
| -rw-r--r-- | tools/perf/util/session.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tools/perf/util/session.c b/tools/perf/util/session.c index ae62d5c9889f..d0053618f540 100644 --- a/tools/perf/util/session.c +++ b/tools/perf/util/session.c @@ -2676,7 +2676,7 @@ bool perf_session__has_switch_events(struct perf_session *session) int map__set_kallsyms_ref_reloc_sym(struct map *map, const char *symbol_name, u64 addr) { - char *bracket; + char *bracket, *name; struct ref_reloc_sym *ref; struct kmap *kmap; @@ -2684,13 +2684,13 @@ int map__set_kallsyms_ref_reloc_sym(struct map *map, const char *symbol_name, u6 if (ref == NULL) return -ENOMEM; - ref->name = strdup(symbol_name); + ref->name = name = strdup(symbol_name); if (ref->name == NULL) { free(ref); return -ENOMEM; } - bracket = strchr(ref->name, ']'); + bracket = strchr(name, ']'); if (bracket) *bracket = '\0'; |