Project

General

Profile

Actions

Bug #8922

closed

Static routes set by system.inc for DNS gateway bindings are not removed

Added by → luckman212 over 5 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
Normal
Category:
Routing
Target version:
Start date:
09/20/2018
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
2.4.4
Affected Architecture:

Description

I found some bugs with system.inc and system.php while troubleshooting a recent issue.

quick repro (2 different bugs):

first
1. go to System > General
2. click 'add dns server' button
3. enter 5.6.7.8 and choose a gateway
4. save
5. delete 5.6.7.8
6. save
static route for 5.6.7.8 is still present on your system, with no way to remove it
(you can confirm with `netstat -4rn`)

second
1. go to System > General
2. add dns server: 1.2.3.4
3. add dns server 2.3.4.5
(you can choose gateways or leave 'none')
4. save
5. delete those 2 dns servers
6. save
<dns2gw> and <dns3gw> remain in the config.xml (orphans)

I pushed this commit that fixes these as well as some other minor changes to make System > General work better (more detail on the PR)
https://github.com/pfsense/pfsense/commit/f211d7c0dcd75369c8f51765e23e7752f593b9af

tested on latest 2.4.4 snaps on both a sg-3100 and sg-4860

Actions #2

Updated by Jim Pingle over 4 years ago

  • Status changed from New to Pull Request Review
Actions #3

Updated by Renato Botelho over 4 years ago

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

PR has been merged. Thanks!

Actions #4

Updated by Viktor Gurov about 4 years ago

  • Status changed from Feedback to Resolved

Renato Botelho wrote:

PR has been merged. Thanks!

tested on 2.5.0.a.20200109.0836

both bugs are resolved

Actions

Also available in: Atom PDF