diff options
| author | Stephen Hemminger <stephen@networkplumber.org> | 2024-01-27 13:58:14 -0800 |
|---|---|---|
| committer | Stephen Hemminger <stephen@networkplumber.org> | 2024-01-27 13:59:49 -0800 |
| commit | 69e3b2fadcd32683db2942f31fe41f0fbb2185f8 (patch) | |
| tree | 0f7514df0ef4219230a55fafd7551ce9c9dabcd8 | |
| parent | 0c3400cc8f576b9f9e4099b67ae53596111323cd (diff) | |
| download | iproute2-next-69e3b2fadcd32683db2942f31fe41f0fbb2185f8.tar.gz | |
bpf: fix warning from basename()
The function basename() expects a mutable character string,
which now causes a warning:
bpf_legacy.c: In function ‘bpf_load_common’:
bpf_legacy.c:975:38: warning: passing argument 1 of ‘__xpg_basename’ discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
975 | basename(cfg->object), cfg->mode == EBPF_PINNED ?
| ~~~^~~~~~~~
In file included from bpf_legacy.c:21:
/usr/include/libgen.h:34:36: note: expected ‘char *’ but argument is of type ‘const char *’
34 | extern char *__xpg_basename (char *__path) __THROW;
Fixes: f20ff2f19552 ("bpf: keep parsed program mode in struct bpf_cfg_in")
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
| -rw-r--r-- | lib/bpf_legacy.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/bpf_legacy.c b/lib/bpf_legacy.c index 741eec8d4..c8da4a3e6 100644 --- a/lib/bpf_legacy.c +++ b/lib/bpf_legacy.c @@ -972,8 +972,8 @@ int bpf_load_common(struct bpf_cfg_in *cfg, const struct bpf_cfg_ops *ops, ops->cbpf_cb(nl, cfg->opcodes, cfg->n_opcodes); if (cfg->mode == EBPF_OBJECT || cfg->mode == EBPF_PINNED) { snprintf(annotation, sizeof(annotation), "%s:[%s]", - basename(cfg->object), cfg->mode == EBPF_PINNED ? - "*fsobj" : cfg->section); + basename(strdupa(cfg->object)), + cfg->mode == EBPF_PINNED ? "*fsobj" : cfg->section); ops->ebpf_cb(nl, cfg->prog_fd, annotation); } |