entrouvert
/
univnautes
Archived
17
0
Fork 0
Code Issues Pull Requests Releases Activity

isset($_GET) seems to always evaluate to true, use something more specific. Fixes use of rc.linkup when run from CLI. Others likely fix similar circumstances, though maybe not ones that are used anywhere.

This commit is contained in:
Chris Buechler 2014-11-05 19:47:22 -06:00
parent c75e8aed2f
commit e55e4b748c
4 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
etc/rc.dyndns.update
View File

@ -37,7 +37,7 @@ require_once("shaper.inc");
/* Interface IP address has changed */
if (isset($_GET))
if (isset($_GET['dyndns']))
$argument = $_GET['dyndns'];
else
$argument = trim($argv[1], " \n");

2
etc/rc.interfaces_wan_configure
View File

@ -34,7 +34,7 @@ require_once("functions.inc");
require_once("filter.inc");
require_once("shaper.inc");
if (isset($_GET))
if (isset($_GET['interface']))
$argument = $_GET['interface'];
else
$argument = str_replace("\n", "", $argv[1]);

2
etc/rc.linkup
View File

@ -85,7 +85,7 @@ function handle_argument_group($iface, $argument2) {
global $g;
if (!file_exists("{$g['varrun_path']}/booting") && empty($g['booting'])) {
if (isset($_GET)) {
if (isset($_GET['interface'])) {
if (!empty($_GET['interface']))
handle_argument_group($_GET['interface'], $_GET['action']);
} else {

2
etc/rc.openvpn
View File

@ -71,7 +71,7 @@ if (file_exists("{$g['varrun_path']}/booting"))
return;
/* Input argument is a comma-separated list of gateway names, blank or "all". */
if (isset($_GET))
if (isset($_GET['interface']))
$argument = $_GET['interface'];
else
$argument = trim($argv[1], " \n");