pfSense

Update reserved_keywords checks to match firewall_aliases_edit

firewall_aliases_import should have the same checks for reserved names as firewall_aliases_edit
This code should really be in a function in a common include file, but which one is the appropriate one?...

Fix display of CIDR/Update Freq in Alias Edit

Fixes #3376. I have no idea what the "^" characters were meant to do, but removing them makes the CIDR/Update Freq value be displayed correctly when editing. Will there be some other side-effect from removing the "^"?

Validate IP address ranges correctly on Alias Bulk Import

The code was there to attempt to validate and implement IP address range lines in Alias Bulk Import e.g.
10.20.0.0-10.21.22.0
should produce a bunch of smaller ranges with appropriate CIDRs.
This fixes the code so IP address ranges actually make it through into the resulting Alias.

Remove not needed code

Make sense of interface mtu handling code. No need to do unneeded operations. This fixes slow boot times and proper handling of mtu for vlans though some work or better model is needed for other interface types.

Delete static route when monitor IP is removed, also save monitor IP even when it's disabled

When WANTIME is empty, there is nothing to do here

test only does integer comparison, use bc to compare float

Save status even if no script is executed

On first run REVIOUSSTATUS doesn't exist, so it cannot be UP or DOWN, invert the logic to fix this. While I'm here, check if file exists before cat it