Commit e7ed83d6 authored by Dan Carpenter's avatar Dan Carpenter Committed by Daniel Borkmann

bpf: Fix an error code in check_btf_func()

This code returns success if the "info_aux" allocation fails but it
should return -ENOMEM.

Fixes: 8c1b6e69 ("bpf: Compare BTF types of functions arguments with actual types")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarDaniel Borkmann <daniel@iogearbox.net>
Acked-by: default avatarSong Liu <songliubraving@fb.com>
Link: https://lore.kernel.org/bpf/20200604085436.GA943001@mwanda
parent 49b23575
...@@ -7552,7 +7552,7 @@ static int check_btf_func(struct bpf_verifier_env *env, ...@@ -7552,7 +7552,7 @@ static int check_btf_func(struct bpf_verifier_env *env,
const struct btf *btf; const struct btf *btf;
void __user *urecord; void __user *urecord;
u32 prev_offset = 0; u32 prev_offset = 0;
int ret = 0; int ret = -ENOMEM;
nfuncs = attr->func_info_cnt; nfuncs = attr->func_info_cnt;
if (!nfuncs) if (!nfuncs)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment