diff options
| author |   Andrew Yourtchenko <ayourtch@gmail.com> | 2018-02-08 21:45:08 +0100 |
|---|---|---|
| committer |   John Lo <loj@cisco.com> | 2018-02-08 22:34:29 +0000 |
| commit | 08118f00e113424ca8ae22d83c766b14297a15f8 (patch) | |
| tree | 8728ab2334b37d2c15109901a90b530431c743d9 /src | |
| parent | 97cabc96916997395d49d4ad9fb4037700e90e6b (diff) | |
classifier-based ACL: testcases for L2 ACLs + fix the enabling of outbound L2 ACL
There was no test coverage for the L2 ACL (other than indirect by
means of ACL plugin tests), so the enabling of the outbound ACL
got fumbled throughout the revisions of the refactoring.
Fix both issues - the error and the lack of test coverage for L2 ACL.
Change-Id: Ib7f42780ef84b4a4f70bd88d7319aeeda866cf06
Signed-off-by: Andrew Yourtchenko <ayourtch@gmail.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/vnet/classify/in_out_acl.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/src/vnet/classify/in_out_acl.c b/src/vnet/classify/in_out_acl.c index a34f6cec61a..2d156ff4373 100644 --- a/src/vnet/classify/in_out_acl.c +++ b/src/vnet/classify/in_out_acl.c @@ -28,9 +28,12 @@ vnet_in_out_acl_ip_feature_enable (vlib_main_t * vnm, if (tid == IN_OUT_ACL_TABLE_L2) { - l2input_intf_bitmap_enable (sw_if_index, - is_output ? L2OUTPUT_FEAT_ACL : - L2INPUT_FEAT_ACL, feature_enable); + if (is_output) + l2output_intf_bitmap_enable (sw_if_index, L2OUTPUT_FEAT_ACL, + feature_enable); + else + l2input_intf_bitmap_enable (sw_if_index, L2INPUT_FEAT_ACL, + feature_enable); } else { /* IP[46] */ |