diff options
| author | Xin Liu <[email protected]> | 2022-09-30 17:07:08 +0800 |
|---|---|---|
| committer | Andrii Nakryiko <[email protected]> | 2022-09-30 15:16:22 -0700 |
| commit | 51e05a8cf8eb34da7473823b7f236a77adfef0b4 (patch) | |
| tree | 8f7cb7c5b1a2f5a1017ae2e40e9bb2c54670d93b | |
| parent | 2efcf695bfc0f078dd7d5d23d96a97db34c930d5 (diff) | |
libbpf: Fix overrun in netlink attribute iteration
I accidentally found that a change in commit 1045b03e07d8 ("netlink: fix
overrun in attribute iteration") was not synchronized to the function
`nla_ok` in tools/lib/bpf/nlattr.c, I think it is necessary to modify,
this patch will do it.
Signed-off-by: Xin Liu <[email protected]>
Signed-off-by: Andrii Nakryiko <[email protected]>
Link: https://lore.kernel.org/bpf/[email protected]
| -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 f57e77a6e40f..3900d052ed19 100644 --- a/tools/lib/bpf/nlattr.c +++ b/tools/lib/bpf/nlattr.c @@ -32,7 +32,7 @@ static struct nlattr *nla_next(const struct nlattr *nla, int *remaining) static int nla_ok(const struct nlattr *nla, int remaining) { - return remaining >= sizeof(*nla) && + return remaining >= (int)sizeof(*nla) && nla->nla_len >= sizeof(*nla) && nla->nla_len <= remaining; } |