From bdcff0327db0ad680906462cdf955d5ae5191c69 Mon Sep 17 00:00:00 2001 From: Andrew Yourtchenko Date: Wed, 14 Nov 2018 16:39:39 +0100 Subject: [PATCH] acl-plugin: fix coverity error 188909 in unit-test code The assignment was redundant with a one just a dozen lines above in the case of the ACL loaded being non-empty, so its only apparent purpose in life was make coverity unhappy... Thus fix by deletion. Change-Id: I573308cb9c212bdfdca2551aa381720dbbcb006e Signed-off-by: Andrew Yourtchenko --- src/plugins/acl/acl_test.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/plugins/acl/acl_test.c b/src/plugins/acl/acl_test.c index dcb0888076d..ef98f79351f 100644 --- a/src/plugins/acl/acl_test.c +++ b/src/plugins/acl/acl_test.c @@ -652,7 +652,6 @@ api_acl_add_replace_from_file (vat_main_t * vam) rules[rule_idx].proto = proto; } - rules[rule_idx].is_permit = is_permit; if (append_default_permit) { rule_idx++; -- 2.16.6