Hello, On Wed, Sep 30, 2020 at 11:02:28PM +0200, Klemens Nanni wrote: > On Sun, Sep 20, 2020 at 07:29:38PM +0200, Klemens Nanni wrote: > > Rebased diff after yasouka's pfctl commit; it still takes care of > > rdomains only, but I'd appreciate folks using `on rdomain' in their > > pf.conf test this. If this works out I'd like to put it in shortly > > after release and work on rtables next. > Working for me so far, anyone else playing around with it? > > I also checked CVS log and the existing rtable_l2() check goes back to > claudio's introduction of `on rdomain N' in pf, i.e. it's been there > from the start as a harmless but needless check and has not been added > after the fact to fix anything. > > The time is now, so here's an updated diff after sashan pointed out how > I erroneously checked the rtable ID instead of the rdomain ID in the > ioctl (copy/pasta mistake). > > Feedback? OK?
fix looks OK to me. thanks and regards sashan
