From 7e60499b6cb941547a147ed66f912aee148ff252 Mon Sep 17 00:00:00 2001 From: Anatoly Burakov Date: Thu, 21 Dec 2017 15:53:05 +0000 Subject: [PATCH] test/table: fix uninitialized parameter delete_bulk() copies metadata to pointers provided by the entries parameter, but in the unit test, they are uninitialized, leading to rte_table attempting to memcpy into random garbage pointers. Memsetting pointer table to zero will prevent that from happening. Fixes: 48f2543cf0a8 ("app/test: add bulk adding and deleting") Cc: stable@dpdk.org Signed-off-by: Anatoly Burakov Acked-by: Cristian Dumitrescu --- test/test/test_table_acl.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/test/test/test_table_acl.c b/test/test/test_table_acl.c index e180202f4e..33e2f6ee64 100644 --- a/test/test/test_table_acl.c +++ b/test/test/test_table_acl.c @@ -503,6 +503,8 @@ setup_acl_pipeline(void) struct rte_pipeline_table_entry *table_entries[5]; int key_found[5]; + memset(table_entries, 0, sizeof(table_entries)); + for (n = 0; n < 5; n++) { memset(&keys[n], 0, sizeof(struct rte_table_acl_rule_delete_params)); key_array[n] = &keys[n]; -- 2.20.1