https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=210924

--- Comment #3 from Kristof Provost <k...@freebsd.org> ---
It's probably a little too late to get away with changing the altq keywords.
This has hit 10.3 (and soon 11.0).

It should be possible to teach pfctl that both 'set timeout interval 10' and
the new interval option are valid.

I'm not really good with yacc, but this seems to work, even if it looks a
little ugly:
diff --git a/sbin/pfctl/parse.y b/sbin/pfctl/parse.y
index e0cfa3d..9457a5b 100644
--- a/sbin/pfctl/parse.y
+++ b/sbin/pfctl/parse.y
@@ -4460,6 +4460,19 @@ timeout_spec     : STRING NUMBER
                        }
                        free($1);
                }
+               | INTERVAL NUMBER
+               {
+                       if (check_rulestate(PFCTL_STATE_OPTION)) {
+                               YYERROR;
+                       }
+                       if ($2 < 0 || $2 > UINT_MAX) {
+                               yyerror("only positive values permitted");
+                               YYERROR;
+                       }
+                       if (pfctl_set_timeout(pf, "interval", $2, 0) != 0) {
+                               YYERROR;
+                       }
+               }
                ;

 timeout_list   : timeout_list comma timeout_spec optnl

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
freebsd-pf@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-pf
To unsubscribe, send any mail to "freebsd-pf-unsubscr...@freebsd.org"

Reply via email to