From 6c0ef526daedef0727763e790db17954187db23b Mon Sep 17 00:00:00 2001 From: Franco Fichtner Date: Tue, 25 Apr 2017 07:50:57 +0200 Subject: [PATCH] rc: newwanip scripts have been subject to extensive bitrot Essentially, the scripts do the same thing, which means we can start to align the two, maybe even merge them if we are lucky. ;) --- src/etc/rc.newwanip | 92 ++++++++++++++++++++----------------------- src/etc/rc.newwanipv6 | 17 ++++---- 2 files changed, 51 insertions(+), 58 deletions(-) diff --git a/src/etc/rc.newwanip b/src/etc/rc.newwanip index 9857edfc6..59e8fcf11 100755 --- a/src/etc/rc.newwanip +++ b/src/etc/rc.newwanip @@ -2,40 +2,40 @@ - Copyright (C) 2003-2005 Manuel Kasper - All rights reserved. - - Redistribution and use in source and binary forms, with or without - modification, are permitted provided that the following conditions are met: - - 1. Redistributions of source code must retain the above copyright notice, - this list of conditions and the following disclaimer. - - 2. Redistributions in binary form must reproduce the above copyright - notice, this list of conditions and the following disclaimer in the - documentation and/or other materials provided with the distribution. - - THIS SOFTWARE IS PROVIDED ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, - INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY - AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE - AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, - OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF - SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS - INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN - CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) - ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE - POSSIBILITY OF SUCH DAMAGE. -*/ + * Copyright (C) 2006 Scott Ullrich + * Copyright (C) 2003-2005 Manuel Kasper + * All rights reserved. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions are met: + * + * 1. Redistributions of source code must retain the above copyright notice, + * this list of conditions and the following disclaimer. + * + * 2. Redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in the + * documentation and/or other materials provided with the distribution. + * + * THIS SOFTWARE IS PROVIDED ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, + * INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY + * AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE + * AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, + * OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF + * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS + * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN + * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) + * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE + * POSSIBILITY OF SUCH DAMAGE. + */ require_once("config.inc"); -require_once('auth.inc'); +require_once("auth.inc"); require_once("filter.inc"); +require_once("services.inc"); require_once("rrd.inc"); require_once("util.inc"); require_once("system.inc"); require_once("interfaces.inc"); -require_once("services.inc"); // Do not process while booting if (file_exists('/var/run/booting')) { @@ -53,49 +53,41 @@ log_error("Informational is starting '{$argument}'"); if (empty($argument)) { $interface = "wan"; - $interface_real = get_real_interface(); + $interface_real = get_real_interface($interface); + $curwanip = get_interface_ip($interface); } else { $interface = convert_real_interface_to_friendly_interface_name($argument); $interface_real = $argument; + $curwanip = find_interface_ip($interface_real); + if (empty($curwanip)) { + $curwanip = get_interface_ip($interface); + } } -$interface_descr = convert_friendly_interface_to_friendly_descr($interface); - /* If the interface is configured and not enabled, bail. We do not need to change settings for disabled interfaces. #3313 */ if (!isset($config['interfaces'][$interface]['enable'])) { log_error("Interface is disabled or empty, nothing to do."); return; } -if (empty($argument)) { - $curwanip = get_interface_ip(); -} else { - $curwanip = find_interface_ip($interface_real); - if(empty($curwanip)) { - $curwanip = get_interface_ip($interface); - } -} +$interface_descr = convert_friendly_interface_to_friendly_descr($interface); log_error("On (IP address: {$curwanip}) (interface: {$interface_descr}[{$interface}]) (real interface: {$interface_real})."); /* - * NOTE: Take care of openvpn, no-ip or similar interfaces if you generate the event to reconfigure an interface. - * i.e. OpenVPN might be in tap mode and not have an ip. + * NOTE: Take care of openvpn and similar if you generate the event to reconfigure an interface. + * i.e. OpenVPN might be in tap mode and not have an ip. */ -if ($curwanip == "0.0.0.0" || !is_ipaddr($curwanip)) { - if (substr($interface_real, 0, 4) != "ovpn") { - if (!empty($config['interfaces'][$interface]['ipaddr'])) { - log_error("Failed to update {$interface} IP, restarting..."); - configd_run("interface reconfigure {$interface}"); - return; - } +if ((empty($curwanip) || !is_ipaddr($curwanip)) && substr($interface_real, 0, 4) != "ovpn") { + if (!empty($config['interfaces'][$interface]['ipaddr'])) { + log_error("Failed to update {$interface} IP, restarting..."); + configd_run("interface reconfigure {$interface}"); + return; } } $oldip = @file_get_contents("/var/db/{$interface}_cacheip"); -system_resolvconf_generate(); - /* write the current interface IP to file */ /* used in src/sbin/dhclient-script.ext */ if (is_ipaddr($curwanip)) { @@ -124,7 +116,7 @@ if (!empty($bridgetmp)) { interface_bridge_add_member($bridgetmp, $interface_real); } -/* make new hosts file */ +system_resolvconf_generate(); system_hosts_generate(); /* check tunneled IPv6 interface tracking */ diff --git a/src/etc/rc.newwanipv6 b/src/etc/rc.newwanipv6 index e103ce4dd..b354fc7aa 100755 --- a/src/etc/rc.newwanipv6 +++ b/src/etc/rc.newwanipv6 @@ -3,7 +3,7 @@ /* * Copyright (C) 2006 Scott Ullrich - * Copyright (C) 2003-2005 Manuel Kasper . + * Copyright (C) 2003-2005 Manuel Kasper * All rights reserved. * * Redistribution and use in source and binary forms, with or without @@ -29,7 +29,7 @@ */ require_once("config.inc"); -require_once("interfaces.inc"); +require_once("auth.inc"); require_once("filter.inc"); require_once("services.inc"); require_once("rrd.inc"); @@ -57,21 +57,23 @@ if (empty($argument)) { $curwanipv6 = get_interface_ipv6($interface, true); } -$interface_descr = convert_friendly_interface_to_friendly_descr($interface); - /* If the interface is configured and not enabled, bail. We do not need to change settings for disabled interfaces. #3313 */ if (!isset($config['interfaces'][$interface]['enable'])) { log_error("Interface is disabled or empty, nothing to do."); return; } +$interface_descr = convert_friendly_interface_to_friendly_descr($interface); + +log_error("On (IP address: {$curwanipv6}) (interface: {$interface_descr}[{$interface}]) (real interface: {$interface_real})."); + /* * NOTE: Take care of openvpn and similar if you generate the event to reconfigure an interface. * i.e. OpenVPN might be in tap mode and not have an ip. */ if ((empty($curwanipv6) || !is_ipaddrv6($curwanipv6)) && substr($interface_real, 0, 4) != "ovpn") { - log_error("Failed to detect IPv6 for {$interface_descr}[{$interface}]"); - return; + log_error("Failed to detect IPv6 for {$interface_descr}[{$interface}]"); + return; } $new_domain_name_servers = getenv("new_domain_name_servers"); @@ -97,8 +99,6 @@ if (is_ipaddrv6($curwanipv6)) { @file_put_contents("/var/db/{$interface}_ipv6", $curwanipv6); } -log_error("On (IP address: {$curwanipv6}) (interface: {$interface}) (real interface: {$interface_real})."); - $oldipv6 = @file_get_contents("/var/db/{$interface}_cacheipv6"); $grouptmp = link_interface_to_group($interface); @@ -108,6 +108,7 @@ if (!empty($grouptmp)) { link_interface_to_track6($interface, "update"); system_resolvconf_generate(); +system_hosts_generate(); /* * We need to force sync VPNs on such even when the IP is the same for dynamic interfaces.