I suggest moving the validation up, to the rte_acl_build() level. Validation should happen at the input to avoid dealing with invalid data later. acl_build_rules() can use category mask without checks because it is validated in acl_check_bld_param() --- let's keep validation logic where it is now. Also, acl_build_rules() may be called multiple times and will thus repeat the validation unnecessarily.
- [PATCH] acl: reject rules with inverted rang... Мокрушин Андрей
- Re: [PATCH] acl: reject rules with inve... Konstantin Ananyev
- Re: [PATCH] acl: reject rules with inve... Dmitry Kozlyuk
- [PATCH v2] acl: reject rules with inver... Andrey Mokrushin
- Re: [PATCH v2] acl: reject rules wi... Konstantin Ananyev

