From 78e7b29ff10c234b44f20fe7d031d60748920c2d Mon Sep 17 00:00:00 2001 From: Stephen Hemminger Date: Thu, 10 Feb 2022 15:09:05 -0800 Subject: [PATCH] test/bpf: skip dump if conversion fails If conversion of cBPF to eBPF fails (in rte_bpf_convert) then the test should not try and print the result. Coverity issue: 373661 Fixes: 2eccf6afbea9 ("bpf: add function to convert classic BPF to DPDK BPF") Cc: stable@dpdk.org Signed-off-by: Stephen Hemminger Acked-by: Konstantin Ananyev --- app/test/test_bpf.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/app/test/test_bpf.c b/app/test/test_bpf.c index d1e10adab5..805cce6406 100644 --- a/app/test/test_bpf.c +++ b/app/test/test_bpf.c @@ -3273,8 +3273,10 @@ test_bpf_dump(struct bpf_program *cbf, const struct rte_bpf_prm *prm) printf("cBPF program (%u insns)\n", cbf->bf_len); bpf_dump(cbf, 1); - printf("\neBPF program (%u insns)\n", prm->nb_ins); - rte_bpf_dump(stdout, prm->ins, prm->nb_ins); + if (prm != NULL) { + printf("\neBPF program (%u insns)\n", prm->nb_ins); + rte_bpf_dump(stdout, prm->ins, prm->nb_ins); + } } static int -- 2.39.5