For TEST_TYPE_STATS, worker_pkt_validate() that places frames onto
pkt_buf is not called. Therefore, when dump mode is set, don't call
worker_pkt_dump() for mentioned test type, so that it won't crash on
pkt_buf() access.

Signed-off-by: Maciej Fijalkowski <maciej.fijalkow...@intel.com>
---
 tools/testing/selftests/bpf/xdpxceiver.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/bpf/xdpxceiver.c 
b/tools/testing/selftests/bpf/xdpxceiver.c
index 8b0f7fdd9003..04574c2b4f41 100644
--- a/tools/testing/selftests/bpf/xdpxceiver.c
+++ b/tools/testing/selftests/bpf/xdpxceiver.c
@@ -999,7 +999,7 @@ static void testapp_validate(void)
        pthread_join(t1, NULL);
        pthread_join(t0, NULL);
 
-       if (debug_pkt_dump) {
+       if (debug_pkt_dump && test_type != TEST_TYPE_STATS) {
                worker_pkt_dump();
                for (int iter = 0; iter < num_frames - 1; iter++) {
                        free(pkt_buf[iter]->payload);
-- 
2.20.1

Reply via email to