diff options
author | Ian Denhardt <ian@zenhack.net> | 2021-02-23 21:24:00 -0500 |
---|---|---|
committer | Daniel Borkmann <daniel@iogearbox.net> | 2021-02-26 22:53:50 +0100 |
commit | 85e142cb42a1e7b33971bf035dae432d8670c46b (patch) | |
tree | 68567a605b601b6714ff073af26f0ef1f171e604 /tools/bpf | |
parent | 04883a079968e6250a4549f6fadb6427c534885e (diff) | |
download | linux-85e142cb42a1e7b33971bf035dae432d8670c46b.tar.gz linux-85e142cb42a1e7b33971bf035dae432d8670c46b.tar.bz2 linux-85e142cb42a1e7b33971bf035dae432d8670c46b.zip |
tools, bpf_asm: Exit non-zero on errors
... so callers can correctly detect failure.
Signed-off-by: Ian Denhardt <ian@zenhack.net>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Link: https://lore.kernel.org/bpf/b0bea780bc292f29e7b389dd062f20adc2a2d634.1614201868.git.ian@zenhack.net
Diffstat (limited to 'tools/bpf')
-rw-r--r-- | tools/bpf/bpf_exp.y | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tools/bpf/bpf_exp.y b/tools/bpf/bpf_exp.y index 8d03e5245da5..dfb7254a24e8 100644 --- a/tools/bpf/bpf_exp.y +++ b/tools/bpf/bpf_exp.y @@ -185,13 +185,13 @@ ldx | OP_LDXB number '*' '(' '[' number ']' '&' number ')' { if ($2 != 4 || $9 != 0xf) { fprintf(stderr, "ldxb offset not supported!\n"); - exit(0); + exit(1); } else { bpf_set_curr_instr(BPF_LDX | BPF_MSH | BPF_B, 0, 0, $6); } } | OP_LDX number '*' '(' '[' number ']' '&' number ')' { if ($2 != 4 || $9 != 0xf) { fprintf(stderr, "ldxb offset not supported!\n"); - exit(0); + exit(1); } else { bpf_set_curr_instr(BPF_LDX | BPF_MSH | BPF_B, 0, 0, $6); } } ; @@ -472,7 +472,7 @@ static void bpf_assert_max(void) { if (curr_instr >= BPF_MAXINSNS) { fprintf(stderr, "only max %u insns allowed!\n", BPF_MAXINSNS); - exit(0); + exit(1); } } @@ -522,7 +522,7 @@ static int bpf_find_insns_offset(const char *label) if (ret == -ENOENT) { fprintf(stderr, "no such label \'%s\'!\n", label); - exit(0); + exit(1); } return ret; |