Commit 395e942d authored by Simon Wang's avatar Simon Wang Committed by Daniel Borkmann

bpf: Replace hard-coded 0 with BPF_K in check_alu_op

Enhance readability a bit.
Signed-off-by: default avatarSimon Wang <wangchuanguo@inspur.com>
Signed-off-by: default avatarDaniel Borkmann <daniel@iogearbox.net>
Acked-by: default avatarJohn Fastabend <john.fastabend@gmail.com>
Link: https://lore.kernel.org/bpf/20220622031923.65692-1-wangchuanguo@inspur.com
parent 6dc7a0ba
...@@ -9096,7 +9096,7 @@ static int check_alu_op(struct bpf_verifier_env *env, struct bpf_insn *insn) ...@@ -9096,7 +9096,7 @@ static int check_alu_op(struct bpf_verifier_env *env, struct bpf_insn *insn)
if (opcode == BPF_END || opcode == BPF_NEG) { if (opcode == BPF_END || opcode == BPF_NEG) {
if (opcode == BPF_NEG) { if (opcode == BPF_NEG) {
if (BPF_SRC(insn->code) != 0 || if (BPF_SRC(insn->code) != BPF_K ||
insn->src_reg != BPF_REG_0 || insn->src_reg != BPF_REG_0 ||
insn->off != 0 || insn->imm != 0) { insn->off != 0 || insn->imm != 0) {
verbose(env, "BPF_NEG uses reserved fields\n"); verbose(env, "BPF_NEG uses reserved fields\n");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment