Project

General

Profile

Actions
Copy link

Bug #10716

closed

Policy routing rules are not written correctly for a down gateway

Added by Steve Wheeler almost 4 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Steve Wheeler
Category:
Rules / NAT
Target version:
2.5.0
Start date:
06/30/2020
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
2.4.x
Affected Architecture:
All

Description

There are two options for the behaviour of policy routing rules that reference a gateway that is down.
The default behaviour is to omit the gateway from the rule but the rule itself in every other way remains.

The alternative behaviour, which is selected by enabling 'Skip rules when gateway is down', is to omit the rule entirely from the ruleset when the gateway is down.

Both are currently broken.
In the default mode when the gateway is down due to 100% packet loss it still appears as:

# Gateways
GWGW2 = " route-to ( vtnet0 172.21.16.141 ) "

If it is marked down it appears as:
# Gateways
GWGW2 = "  "

In both cases a rule referencing it still contain the bad gateway, for example:
pass  in  quick  on $LAN  $GWGW2 inet proto tcp  from any to 9.9.9.9 tracker 1593470028 flags S/SA keep state  label "USER_RULE"

In the alternative mode it is different. If the gateway is down due to 100% packet loss it still appears as:

GWGW2 = " route-to ( vtnet0 172.21.16.141 ) "

And the rule referencing it is the same:
pass  in  quick  on $LAN  $GWGW2 inet proto tcp  from any to 9.9.9.9 tracker 1593470028 flags S/SA keep state  label "USER_RULE"

However when the gateway is marked down the gateway no appears inthe ruleset at all and the rule referencing it is replaced with:
# rule  disabled because gateway GW2 is down  label "USER_RULE"

That look like the expected behaviour but it is only triggered by manually marking the gateway down which is of limited use.
I suspect the code added here for that particular case is still in play but all other situations are no longer triggering anything.
https://github.com/pfsense/pfsense/pull/1594

Actions
Copy link
 #1

Updated by Jim Pingle almost 4 years ago

Having $GWGW2 on the rule when the content of $GWGW2 is empty is fine. That effectively means the gateway is not being placed on the rule, it just saves logic by not having to have the test both on the code making the rule and the code making the contents of the gateway macro.

The other parts do appear to be problematic, however.

Might be worth checking if https://github.com/pfsense/pfsense/pull/4336 already fixes this.

Actions
Copy link
 #2

Updated by Viktor Gurov almost 4 years ago

Jim Pingle wrote:

Having $GWGW2 on the rule when the content of $GWGW2 is empty is fine. That effectively means the gateway is not being placed on the rule, it just saves logic by not having to have the test both on the code making the rule and the code making the contents of the gateway macro.

The other parts do appear to be problematic, however.

Might be worth checking if https://github.com/pfsense/pfsense/pull/4336 already fixes this.

This is a different issue/feature

Fix:
https://github.com/pfsense/pfsense/pull/4383

It also fixes fw rule creation for the gateways group (down or force_down state)

Actions
Copy link
 #3

Updated by Steve Wheeler almost 4 years ago

Tested this patch. It corrects the 'Skip rules when gateway is down' setting but not the default 'omit gateway from the rule' setting.

Actions
Copy link
 #4

Updated by Steve Wheeler almost 4 years ago

But it sets the value of the gateway variable to 'empty' as Jim commented above so that's OK.

Actions
Copy link
 #5

Updated by Steve Wheeler almost 4 years ago

Tested the new patch with gateway group rules again 2.4.5p1. Seems to work as expected in all conditions.

Tested gateway: UP, manually marked down, down due 100% loss, down due to high packet loss in a group, down due to high latency in a group.

Actions
Copy link
 #6

Updated by Jim Pingle almost 4 years ago

  • Status changed from New to Pull Request Review
Actions
Copy link
 #7

Updated by Renato Botelho almost 4 years ago

  • Status changed from Pull Request Review to Feedback
  • Assignee set to Renato Botelho
  • % Done changed from 0 to 100

PR has been merged. Thanks!

Actions
Copy link
 #8

Updated by Anonymous over 3 years ago

  • Assignee changed from Renato Botelho to Steve Wheeler

Please confirm and resolve

Actions
Copy link
 #9

Updated by Steve Wheeler over 3 years ago

  • Status changed from Feedback to Resolved

This now correctly sets the gateway to none or disables rules entirely on packet loss:

GWT70 = "  " 
...
pass  in  quick  on $LAN  $GWT70 inet proto tcp  from 192.168.167.0/24 to 192.168.226.0/24 tracker 1603033882 flags S/SA keep state  label "USER_RULE: Test Policy rule" 


# rule Test Policy rule disabled because gateway T70 is down  label "USER_RULE: Test Policy rule"

Tested in:

2.5.0-DEVELOPMENT (amd64)
built on Wed Oct 14 13:07:12 EDT 2020
FreeBSD 12.2-STABLE

Actions
Copy link

Also available in: Atom PDF