aboutsummaryrefslogtreecommitdiffstats
path: root/tools/bpf/bpftool/prog.c
diff options
context:
space:
mode:
authorJiong Wang <jiong.wang@netronome.com>2018-03-01 21:01:16 -0500
committerAlexei Starovoitov <ast@kernel.org>2018-03-01 21:29:48 -0500
commit3197239d24dcecef1dbc260733b377ade731b748 (patch)
tree7c21cb31ba8a57160bbb32c7cd488298353ce102 /tools/bpf/bpftool/prog.c
parentc8745e07d5d676b2cb49f90baec1ac5e66cb7383 (diff)
tools: bpftool: remove unnecessary 'if' to reduce indentation
It is obvious we could use 'else if' instead of start a new 'if' in the touched code. Signed-off-by: Jiong Wang <jiong.wang@netronome.com> Acked-by: Jakub Kicinski <jakub.kicinski@netronome.com> Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'tools/bpf/bpftool/prog.c')
-rw-r--r--tools/bpf/bpftool/prog.c38
1 files changed, 18 insertions, 20 deletions
diff --git a/tools/bpf/bpftool/prog.c b/tools/bpf/bpftool/prog.c
index e549e329be82..950d11dd42ab 100644
--- a/tools/bpf/bpftool/prog.c
+++ b/tools/bpf/bpftool/prog.c
@@ -777,27 +777,25 @@ static int do_dump(int argc, char **argv)
777 777
778 if (json_output) 778 if (json_output)
779 jsonw_null(json_wtr); 779 jsonw_null(json_wtr);
780 } else { 780 } else if (member_len == &info.jited_prog_len) {
781 if (member_len == &info.jited_prog_len) { 781 const char *name = NULL;
782 const char *name = NULL; 782
783 783 if (info.ifindex) {
784 if (info.ifindex) { 784 name = ifindex_to_bfd_name_ns(info.ifindex,
785 name = ifindex_to_bfd_name_ns(info.ifindex, 785 info.netns_dev,
786 info.netns_dev, 786 info.netns_ino);
787 info.netns_ino); 787 if (!name)
788 if (!name) 788 goto err_free;
789 goto err_free;
790 }
791
792 disasm_print_insn(buf, *member_len, opcodes, name);
793 } else {
794 kernel_syms_load(&dd);
795 if (json_output)
796 dump_xlated_json(&dd, buf, *member_len, opcodes);
797 else
798 dump_xlated_plain(&dd, buf, *member_len, opcodes);
799 kernel_syms_destroy(&dd);
800 } 789 }
790
791 disasm_print_insn(buf, *member_len, opcodes, name);
792 } else {
793 kernel_syms_load(&dd);
794 if (json_output)
795 dump_xlated_json(&dd, buf, *member_len, opcodes);
796 else
797 dump_xlated_plain(&dd, buf, *member_len, opcodes);
798 kernel_syms_destroy(&dd);
801 } 799 }
802 800
803 free(buf); 801 free(buf);