diff options
author | Jeremy Cline <jeremy@jcline.org> | 2018-02-19 20:00:07 -0500 |
---|---|---|
committer | Daniel Borkmann <daniel@iogearbox.net> | 2018-02-20 15:08:20 -0500 |
commit | b1a2ce825737b0165cc08e6f98f8c0ea1affdd60 (patch) | |
tree | b5b9b483a0803953aaa1c490cc3186c4d150599b /tools/lib/bpf | |
parent | 9c2d63b843a5c8a8d0559cc067b5398aa5ec3ffc (diff) |
tools/libbpf: Avoid possibly using uninitialized variable
Fixes a GCC maybe-uninitialized warning introduced by 48cca7e44f9f.
"text" is only initialized inside the if statement so only print debug
info there.
Fixes: 48cca7e44f9f ("libbpf: add support for bpf_call")
Signed-off-by: Jeremy Cline <jeremy@jcline.org>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Diffstat (limited to 'tools/lib/bpf')
-rw-r--r-- | tools/lib/bpf/libbpf.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tools/lib/bpf/libbpf.c b/tools/lib/bpf/libbpf.c index 97073d649c1a..5bbbf285af74 100644 --- a/tools/lib/bpf/libbpf.c +++ b/tools/lib/bpf/libbpf.c | |||
@@ -1060,11 +1060,12 @@ bpf_program__reloc_text(struct bpf_program *prog, struct bpf_object *obj, | |||
1060 | prog->insns = new_insn; | 1060 | prog->insns = new_insn; |
1061 | prog->main_prog_cnt = prog->insns_cnt; | 1061 | prog->main_prog_cnt = prog->insns_cnt; |
1062 | prog->insns_cnt = new_cnt; | 1062 | prog->insns_cnt = new_cnt; |
1063 | pr_debug("added %zd insn from %s to prog %s\n", | ||
1064 | text->insns_cnt, text->section_name, | ||
1065 | prog->section_name); | ||
1063 | } | 1066 | } |
1064 | insn = &prog->insns[relo->insn_idx]; | 1067 | insn = &prog->insns[relo->insn_idx]; |
1065 | insn->imm += prog->main_prog_cnt - relo->insn_idx; | 1068 | insn->imm += prog->main_prog_cnt - relo->insn_idx; |
1066 | pr_debug("added %zd insn from %s to prog %s\n", | ||
1067 | text->insns_cnt, text->section_name, prog->section_name); | ||
1068 | return 0; | 1069 | return 0; |
1069 | } | 1070 | } |
1070 | 1071 | ||