acl-plugin: assertion failed at hash_lookup.c:226 when modifying ACLs applied as... 51/7651/2
authorAndrew Yourtchenko <ayourtch@gmail.com>
Wed, 19 Jul 2017 17:23:59 +0000 (13:23 -0400)
committerDave Barach <openvpp@barachs.net>
Thu, 20 Jul 2017 12:05:36 +0000 (12:05 +0000)
commit45fe7399152f5ca511ba0b03fee3d5a3dffd1897
tree1ae88f3cdd548a479e2d4b5aee6b50fee62b6b75
parentf4f635e7c05a4430e834a725f611cb83a9179146
acl-plugin: assertion failed at hash_lookup.c:226 when modifying ACLs applied as part of many (VPP-910)

change 7385 has added the code which has the first ACE's "prev" entry within the linked list of
shadowed ACEs pointing to the last ACE, in order to avoid the frequent linear list traversal.
That change was not complete and did not update this "prev" entry whenever the last ACE was deleted.
As a result the changes within the applied ACLs which caused the calls to hash_acl_unapply/hash_acl_apply
may result in hitting assert which does the sanity check. The solution is to add the missing update logic.

Change-Id: I9cbe9a7c68b92fa3a22a8efd11b679667d38f186
Signed-off-by: Andrew Yourtchenko <ayourtch@gmail.com>
src/plugins/acl/hash_lookup.c