Thread (3 messages) 3 messages, 3 authors, 2020-11-24

Re: [PATCH 1/1] tools/bpftool: fix error return value in build_btf_type_table()

From: Yonghong Song <hidden>
Date: 2020-11-24 16:00:35
Also in: bpf, lkml


On 11/24/20 2:41 AM, Zhen Lei wrote:
An appropriate return value should be set on the failed path.

Reported-by: Hulk Robot <redacted>
Signed-off-by: Zhen Lei <redacted>
LGTM.

Acked-by: Yonghong Song <redacted>

Also this is a bug fix. It should probably be targeted to bpf tree. So,

Fixes: 4d374ba0bf30 ("tools: bpftool: implement "bpftool btf show|list"")
quoted hunk ↗ jump to hunk
---
  tools/bpf/bpftool/btf.c | 1 +
  1 file changed, 1 insertion(+)
diff --git a/tools/bpf/bpftool/btf.c b/tools/bpf/bpftool/btf.c
index 8ab142ff5eac..2afb7d5b1aca 100644
--- a/tools/bpf/bpftool/btf.c
+++ b/tools/bpf/bpftool/btf.c
@@ -693,6 +693,7 @@ build_btf_type_table(struct btf_attach_table *tab, enum bpf_obj_type type,
  		obj_node = calloc(1, sizeof(*obj_node));
  		if (!obj_node) {
  			p_err("failed to allocate memory: %s", strerror(errno));
+			err = -ENOMEM;
  			goto err_free;
  		}
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help