diff options
author | Wei Yongjun <weiyongjun1@huawei.com> | 2019-08-16 02:40:44 +0000 |
---|---|---|
committer | Alexei Starovoitov <ast@kernel.org> | 2019-08-15 22:18:17 -0700 |
commit | e03250061b54041d3502696db421c44a4a8039f4 (patch) | |
tree | cd398ab0d2a6f2c747b5af55f32ccf993aa0d042 /kernel/bpf | |
parent | 82c4c3b7c76720672b84c849ffe7367801278b26 (diff) | |
download | linux-e03250061b54041d3502696db421c44a4a8039f4.tar.bz2 |
btf: fix return value check in btf_vmlinux_init()
In case of error, the function kobject_create_and_add() returns NULL
pointer not ERR_PTR(). The IS_ERR() test in the return value check
should be replaced with NULL test.
Fixes: 341dfcf8d78e ("btf: expose BTF info through sysfs")
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Acked-by: Andrii Nakryiko <andriin@fb.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'kernel/bpf')
-rw-r--r-- | kernel/bpf/sysfs_btf.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/kernel/bpf/sysfs_btf.c b/kernel/bpf/sysfs_btf.c index 4659349fc795..7ae5dddd1fe6 100644 --- a/kernel/bpf/sysfs_btf.c +++ b/kernel/bpf/sysfs_btf.c @@ -30,17 +30,12 @@ static struct kobject *btf_kobj; static int __init btf_vmlinux_init(void) { - int err; - if (!_binary__btf_vmlinux_bin_start) return 0; btf_kobj = kobject_create_and_add("btf", kernel_kobj); - if (IS_ERR(btf_kobj)) { - err = PTR_ERR(btf_kobj); - btf_kobj = NULL; - return err; - } + if (!btf_kobj) + return -ENOMEM; bin_attr_btf_vmlinux.size = _binary__btf_vmlinux_bin_end - _binary__btf_vmlinux_bin_start; |