Re: pfctl check if rdomain exists

2018-05-13 Thread Alexander Bluhm
On Sun, May 13, 2018 at 03:22:39PM +0200, Sebastian Benoit wrote: > Here is a better version that moves the RT_TABLEID_MAX check into the > rdomain_check() function. I think hiding the yyerror() in rdomain_check() is not so nice. So I like your first version more. Could it be fixed with an "else

Re: pfctl check if rdomain exists

2018-05-13 Thread Sebastian Benoit
Alexander Bluhm(alexander.bl...@gmx.net) on 2018.05.13 10:03:43 +0200: > On Sun, May 13, 2018 at 01:34:48AM +0200, Sebastian Benoit wrote: > > when you add a pf rule with a "on rdomain n" with nonexisting rdomain n, > > the load will fail with the error > > > > pfctl: DIOCADDRULE: Device busy >

Re: pfctl check if rdomain exists

2018-05-13 Thread Alexander Bluhm
On Sun, May 13, 2018 at 01:34:48AM +0200, Sebastian Benoit wrote: > when you add a pf rule with a "on rdomain n" with nonexisting rdomain n, > the load will fail with the error > > pfctl: DIOCADDRULE: Device busy > > with no information which rule caused the problem and no indication that the >

pfctl check if rdomain exists

2018-05-12 Thread Sebastian Benoit
when you add a pf rule with a "on rdomain n" with nonexisting rdomain n, the load will fail with the error pfctl: DIOCADDRULE: Device busy with no information which rule caused the problem and no indication that the problem is the rdomain . So lets check if the rdomain really exists when parsi