diff --git a/src/firewall/core/rich.py b/src/firewall/core/rich.py index eb4a2d2..86c0c99 100644 --- a/src/firewall/core/rich.py +++ b/src/firewall/core/rich.py @@ -382,7 +382,7 @@ class Rich_Rule(object): try: self.priority = int(attr_value) except ValueError: - raise FirewallError(errors.INVALID_RULE, "invalid 'priority' attribute value '%s'." % attr_value) + raise FirewallError(errors.INVALID_PRIORITY, "invalid 'priority' attribute value '%s'." % attr_value) elif attr_name: if attr_name == 'protocol': err_msg = "wrong 'protocol' usage. Use either 'rule protocol value=...' or 'rule [forward-]port protocol=...'." diff --git a/src/tests/regression/rhbz1689429.at b/src/tests/regression/rhbz1689429.at index 5701607..9157c95 100644 --- a/src/tests/regression/rhbz1689429.at +++ b/src/tests/regression/rhbz1689429.at @@ -1,11 +1,11 @@ FWD_START_TEST([rich rule invalid priority]) AT_KEYWORDS(rich rhbz1689429) -FWD_CHECK([--add-rich-rule='rule priority=foo accept'], 122, [], - [Error: INVALID_RULE: invalid 'priority' attribute value 'foo'. +FWD_CHECK([--add-rich-rule='rule priority=foo accept'], 139, [], + [Error: INVALID_PRIORITY: invalid 'priority' attribute value 'foo'. ]) -FWD_CHECK([--permanent --add-rich-rule='rule priority=foo accept'], 122, [], - [Error: INVALID_RULE: invalid 'priority' attribute value 'foo'. +FWD_CHECK([--permanent --add-rich-rule='rule priority=foo accept'], 139, [], + [Error: INVALID_PRIORITY: invalid 'priority' attribute value 'foo'. ]) FWD_RELOAD