diff options
author | Toke Høiland-Jørgensen <toke@redhat.com> | 2019-11-09 21:37:30 +0100 |
---|---|---|
committer | Alexei Starovoitov <ast@kernel.org> | 2019-11-10 19:26:30 -0800 |
commit | b6e99b010ecf829fd8453a7a77e389501bb81990 (patch) | |
tree | e4113e360fc202b4c7e663f28fa8e6c661c025e9 /tools/lib/bpf/netlink.c | |
parent | 4f33ddb4e3e28ade924fbf05ec8bcd7d5c022fee (diff) | |
download | linux-b6e99b010ecf829fd8453a7a77e389501bb81990.tar.gz linux-b6e99b010ecf829fd8453a7a77e389501bb81990.tar.bz2 linux-b6e99b010ecf829fd8453a7a77e389501bb81990.zip |
libbpf: Use pr_warn() when printing netlink errors
The netlink functions were using fprintf(stderr, ) directly to print out
error messages, instead of going through the usual logging macros. This
makes it impossible for the calling application to silence or redirect
those error messages. Fix this by switching to pr_warn() in nlattr.c and
netlink.c.
Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Acked-by: Andrii Nakryiko <andriin@fb.com>
Acked-by: David S. Miller <davem@davemloft.net>
Acked-by: Song Liu <songliubraving@fb.com>
Link: https://lore.kernel.org/bpf/157333185055.88376.15999360127117901443.stgit@toke.dk
Diffstat (limited to 'tools/lib/bpf/netlink.c')
-rw-r--r-- | tools/lib/bpf/netlink.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tools/lib/bpf/netlink.c b/tools/lib/bpf/netlink.c index ce3ec81b71c0..a261df9cb488 100644 --- a/tools/lib/bpf/netlink.c +++ b/tools/lib/bpf/netlink.c @@ -12,6 +12,7 @@ #include "bpf.h" #include "libbpf.h" +#include "libbpf_internal.h" #include "nlattr.h" #ifndef SOL_NETLINK @@ -43,7 +44,7 @@ int libbpf_netlink_open(__u32 *nl_pid) if (setsockopt(sock, SOL_NETLINK, NETLINK_EXT_ACK, &one, sizeof(one)) < 0) { - fprintf(stderr, "Netlink error reporting not supported\n"); + pr_warn("Netlink error reporting not supported\n"); } if (bind(sock, (struct sockaddr *)&sa, sizeof(sa)) < 0) { |