documentation/3270: PF BNF grammar has errors (or so it seems)

>Number: 3270
>Category: documentation
>Synopsis: PF BNF grammar has errors (or so it seems)
>Confidential: no
net
>Environment:
        

	System      : OpenBSD 3.3
	Architecture: OpenBSD.i386
	Machine     : i386

In the BNF grammar of PF (man 5 pf.conf) we define the following:

        subqueue = STRING | '{' QUEUE-LIST '}'

But this is never referenced from anywhere else in the grammar.

In the PF faq we define:

altq on dc0 cbq bandwidth 2Mb queue { std_in, ssh_im_in, dns_in, bob_in }

Which according to the grammar must be:

altq-rule      = altq on interface-name queueopts-list
                      queue queue-list

But queue list is defined as:

Do you need help?

In addition to our free technical library, Pantek provides professional IT services to companies who utilize Open Source software: Learn more about Pantek's Technical Support offerings Place an order for 24/7/365 Technical Support here Learn more about Pantek, who we are, and what we do Chat with a Live Operator Related openbsd.org Links advocacy announce bugs ipv6 misc ports ports-changes security-announce smp source-changes tech Request more information about Pantek

queue-list = string [ [ , ] string ]

So the above Faq line does not abide the grammar (or the other way around ;)

/usr/src/sbin/pfctl/parse.y states:

...
altqif : ALTQ interface queue_opts QUEUE qassign { ...

and

qassign         : /* empty */           { $$ = NULL; }
                | qassign_item          { $$ = $1; }
                | '{' qassign_list '}'  { $$ = $2; }
                ;

which is further defined as you might expect. I do not know what the most beautiful solution is, but it is definitely not correct grammar.

>How-To-Repeat:

        man 5 pf.conf - read
>Fix:

Do you need more help?

In addition to our free technical library, Pantek provides professional IT services to companies who utilize Open Source software: Learn more about Pantek's Technical Support offerings Place an order for 24/7/365 Technical Support here Learn more about Pantek, who we are, and what we do Chat with a Live Operator Related openbsd.org Links advocacy announce bugs ipv6 misc ports ports-changes security-announce smp source-changes tech Request more information about Pantek

        read the source-code and see what the grammar does in reality.

>Release-Note:
Received on Thu May 22 08:09:53 2003