Re: [PATCH bpf-next 2/2] libbpf: Return err if bpf_object__load failed

From: Andrii Nakryiko
Date: Wed Apr 22 2020 - 19:27:19 EST


On Wed, Apr 22, 2020 at 1:30 AM Mao Wenan <maowenan@xxxxxxxxxx> wrote:
>
> bpf_object__load() has various return code, when
> it failed to load object, it must return err instead
> of return -EINVAL.
>
> Signed-off-by: Mao Wenan <maowenan@xxxxxxxxxx>
> ---

This patch looks good. The other one in this series - not so sure..

Acked-by: Andrii Nakryiko <andriin@xxxxxx>

> tools/lib/bpf/libbpf.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/lib/bpf/libbpf.c b/tools/lib/bpf/libbpf.c
> index 8f480e29a6b0..8e1dc6980fac 100644
> --- a/tools/lib/bpf/libbpf.c
> +++ b/tools/lib/bpf/libbpf.c
> @@ -7006,7 +7006,7 @@ int bpf_prog_load_xattr(const struct bpf_prog_load_attr *attr,
> err = bpf_object__load(obj);
> if (err) {
> bpf_object__close(obj);
> - return -EINVAL;
> + return err;
> }
>
> *pobj = obj;
> --
> 2.20.1
>