diff options
| author | Ilya Leoshkevich | 2023-02-10 01:12:01 +0100 |
|---|---|---|
| committer | Andrii Nakryiko | 2023-02-10 15:27:22 -0800 |
| commit | 17bcd27a08a21397698edf143084d7c87ce17946 (patch) | |
| tree | 7e28ab4b11d2a2b09dac570db9adad2bffaea2da /tools | |
| parent | 202702e890a41412a7de970b84a970ba1d5001c9 (diff) | |
libbpf: Fix alen calculation in libbpf_nla_dump_errormsg()
The code assumes that everything that comes after nlmsgerr are nlattrs.
When calculating their size, it does not account for the initial
nlmsghdr. This may lead to accessing uninitialized memory.
Fixes: bbf48c18ee0c ("libbpf: add error reporting in XDP")
Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Link: https://lore.kernel.org/bpf/20230210001210.395194-8-iii@linux.ibm.com
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/lib/bpf/nlattr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/lib/bpf/nlattr.c b/tools/lib/bpf/nlattr.c index 3900d052ed19..975e265eab3b 100644 --- a/tools/lib/bpf/nlattr.c +++ b/tools/lib/bpf/nlattr.c @@ -178,7 +178,7 @@ int libbpf_nla_dump_errormsg(struct nlmsghdr *nlh) hlen += nlmsg_len(&err->msg); attr = (struct nlattr *) ((void *) err + hlen); - alen = nlh->nlmsg_len - hlen; + alen = (void *)nlh + nlh->nlmsg_len - (void *)attr; if (libbpf_nla_parse(tb, NLMSGERR_ATTR_MAX, attr, alen, extack_policy) != 0) { |