From d2193f76caf8d49ab27d8207de578db5de4fa67f Mon Sep 17 00:00:00 2001
From: Tony Murray <murraytony@gmail.com>
Date: Wed, 14 Sep 2016 10:53:04 -0500
Subject: [PATCH] feature: allow scripts to be run from any working directory
 (#4437)

* feature: allow validate.php to be run from any working directory

* remove redundant realpath() call

* re-add realpath() to remove symbolic links

* realpath() isn't needed...

* chdir() in all php scripts in ./ and ./scripts/

* update-sql.php clearly is not used, as it was broken.

* Change some scripts to executable
Remove extra chdir() call in snmp-scan.php

* Missed console-ui.php
Kind of fixed console-ui.php help output

* Re-add newline
---
 addhost.php               |  2 +-
 adduser.php               |  2 +-
 alerts.php                |  2 ++
 billing-calculate.php     |  2 +-
 build-base.php            |  3 +++
 check-services.php        |  2 +-
 config_to_json.php        | 24 ++++++----------------
 daily.php                 |  3 +++
 delhost.php               |  2 +-
 discovery.php             |  2 +-
 dist-pollers.php          |  2 +-
 irc.php                   |  3 ++-
 poll-billing.php          |  2 +-
 poller.php                |  2 +-
 renamehost.php            |  2 +-
 scripts/console-ui.php    | 42 +++++++++++++++++++--------------------
 scripts/gen_smokeping.php |  8 +++++---
 scripts/tune_port.php     |  4 ++++
 scripts/update-sql.php    | 22 --------------------
 snmp-scan.php             |  4 ++--
 snmptrap.php              |  2 ++
 syslog.php                |  2 ++
 validate.php              |  6 ++++--
 23 files changed, 65 insertions(+), 80 deletions(-)
 mode change 100644 => 100755 build-base.php
 mode change 100644 => 100755 config_to_json.php
 delete mode 100755 scripts/update-sql.php

diff --git a/addhost.php b/addhost.php
index ab69ea2784c5..d7edc16ce921 100755
--- a/addhost.php
+++ b/addhost.php
@@ -13,7 +13,7 @@
 
 use LibreNMS\Exceptions\HostUnreachableException;
 
-chdir(dirname($argv[0]));
+chdir(__DIR__); // cwd to the directory containing this script
 
 require 'includes/defaults.inc.php';
 require 'config.php';
diff --git a/adduser.php b/adduser.php
index 8ba8431ad174..a0a31d2bde79 100755
--- a/adduser.php
+++ b/adduser.php
@@ -12,7 +12,7 @@
  *
  */
 
-chdir(dirname($argv[0]));
+chdir(__DIR__); // cwd to the directory containing this script
 
 require 'includes/defaults.inc.php';
 require 'config.php';
diff --git a/alerts.php b/alerts.php
index 143a6fcb0c77..345dc7fd1af2 100755
--- a/alerts.php
+++ b/alerts.php
@@ -25,6 +25,8 @@
  * @subpackage Alerts
  */
 
+chdir(__DIR__); // cwd to the directory containing this script
+
 require_once 'includes/defaults.inc.php';
 require_once 'config.php';
 
diff --git a/billing-calculate.php b/billing-calculate.php
index 243d7fdfc253..90058a0902c5 100755
--- a/billing-calculate.php
+++ b/billing-calculate.php
@@ -11,7 +11,7 @@
  * @copyright  (C) 2006 - 2012 Adam Armstrong
  */
 
-chdir(dirname($argv[0]));
+chdir(__DIR__); // cwd to the directory containing this script
 
 require 'includes/defaults.inc.php';
 require 'config.php';
diff --git a/build-base.php b/build-base.php
old mode 100644
new mode 100755
index 3256594a8faa..b1d0bdfec21e
--- a/build-base.php
+++ b/build-base.php
@@ -1,5 +1,8 @@
+#!/usr/bin/env php
 <?php
 
+chdir(__DIR__); // cwd to the directory containing this script
+
 // MYSQL Check - FIXME
 // 1 UNKNOWN
 include 'config.php';
diff --git a/check-services.php b/check-services.php
index ee2ab28a6426..fd2cf0ddcfc2 100755
--- a/check-services.php
+++ b/check-services.php
@@ -13,7 +13,7 @@
  * the source code distribution for details.
  */
 
-chdir(dirname($argv[0]));
+chdir(__DIR__); // cwd to the directory containing this script
 
 require 'includes/defaults.inc.php';
 require 'config.php';
diff --git a/config_to_json.php b/config_to_json.php
old mode 100644
new mode 100755
index 789e3b9686e0..c183be1a4aeb
--- a/config_to_json.php
+++ b/config_to_json.php
@@ -1,3 +1,4 @@
+#!/usr/bin/env php
 <?php
 
 /*
@@ -6,24 +7,11 @@
  *
  */
 
-$defaults_file = 'includes/defaults.inc.php';
-$config_file   = 'config.php';
-
-// move to install dir
-chdir(dirname($argv[0]));
-
-function iscli()
-{
-    if (php_sapi_name() == 'cli' && empty($_SERVER['REMOTE_ADDR'])) {
-        return true;
-    } else {
-        return false;
-    }
-}
+chdir(__DIR__); // cwd to the directory containing this script
 
 // check if we are running through the CLI, otherwise abort
-if (iscli()) {
-    include_once $defaults_file;
-    include_once $config_file;
-    print (json_encode($config));
+if (php_sapi_name() == 'cli' && empty($_SERVER['REMOTE_ADDR'])) {
+    include_once 'includes/defaults.inc.php';
+    include_once 'config.php';
+    echo json_encode($config);
 }
diff --git a/daily.php b/daily.php
index a18d4525d6d8..df29e7be123f 100644
--- a/daily.php
+++ b/daily.php
@@ -1,3 +1,4 @@
+#!/usr/bin/env php
 <?php
 
 /*
@@ -5,6 +6,8 @@
  * (c) 2013 LibreNMS Contributors
  */
 
+chdir(__DIR__); // cwd to the directory containing this script
+
 require 'includes/defaults.inc.php';
 require 'config.php';
 require_once 'includes/definitions.inc.php';
diff --git a/delhost.php b/delhost.php
index 6b60ef753d70..490acc048070 100755
--- a/delhost.php
+++ b/delhost.php
@@ -11,7 +11,7 @@
  * @copyright  (C) 2006 - 2012 Adam Armstrong
  */
 
-chdir(dirname($argv[0]));
+chdir(__DIR__); // cwd to the directory containing this script
 
 require 'includes/defaults.inc.php';
 require 'config.php';
diff --git a/discovery.php b/discovery.php
index 0a707d982ccf..b3ad5fa0b662 100755
--- a/discovery.php
+++ b/discovery.php
@@ -11,7 +11,7 @@
  * @copyright  (C) 2006 - 2012 Adam Armstrong
  */
 
-chdir(dirname($argv[0]));
+chdir(__DIR__); // cwd to the directory containing this script
 
 require 'includes/defaults.inc.php';
 require 'config.php';
diff --git a/dist-pollers.php b/dist-pollers.php
index 24f314a32c03..198105d73c81 100755
--- a/dist-pollers.php
+++ b/dist-pollers.php
@@ -13,7 +13,7 @@
  * the source code distribution for details.
  */
 
-chdir(dirname($argv[0]));
+chdir(__DIR__); // cwd to the directory containing this script
 
 require 'includes/defaults.inc.php';
 require 'config.php';
diff --git a/irc.php b/irc.php
index 5b78388df296..4b548df1cc22 100755
--- a/irc.php
+++ b/irc.php
@@ -19,7 +19,8 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  * */
 
-chdir(dirname($argv[0]));
+chdir(__DIR__); // cwd to the directory containing this script
+
 require_once 'includes/defaults.inc.php';
 require_once 'config.php';
 require_once 'includes/definitions.inc.php';
diff --git a/poll-billing.php b/poll-billing.php
index d51081057328..50e9c67cac11 100755
--- a/poll-billing.php
+++ b/poll-billing.php
@@ -11,7 +11,7 @@
  * @copyright  (C) 2006 - 2012 Adam Armstrong
  */
 
-chdir(dirname($argv[0]));
+chdir(__DIR__); // cwd to the directory containing this script
 
 // FIXME - implement cli switches, debugging, etc.
 require 'includes/defaults.inc.php';
diff --git a/poller.php b/poller.php
index 41c25ef39afc..92c3687c0c2b 100755
--- a/poller.php
+++ b/poller.php
@@ -11,7 +11,7 @@
  * @copyright  (C) 2006 - 2012 Adam Armstrong
  */
 
-chdir(dirname($argv[0]));
+chdir(__DIR__); // cwd to the directory containing this script
 
 require 'includes/defaults.inc.php';
 require 'config.php';
diff --git a/renamehost.php b/renamehost.php
index 49b3929ed4b8..74475edfc63c 100755
--- a/renamehost.php
+++ b/renamehost.php
@@ -11,7 +11,7 @@
  * @copyright  (C) 2006 - 2012 Adam Armstrong
  */
 
-chdir(dirname($argv[0]));
+chdir(__DIR__); // cwd to the directory containing this script
 
 require 'includes/defaults.inc.php';
 require 'config.php';
diff --git a/scripts/console-ui.php b/scripts/console-ui.php
index ede19c8ecfd1..fba009203767 100755
--- a/scripts/console-ui.php
+++ b/scripts/console-ui.php
@@ -1,11 +1,13 @@
 #!/usr/bin/env php
 <?php
 
-require '../includes/defaults.inc.php';
-require '../config.php';
-require_once '../includes/definitions.inc.php';
-require '../includes/functions.php';
-require '../html/includes/functions.inc.php';
+chdir(realpath(__DIR__ . '/..')); // cwd to the parent directory of this script
+
+require 'includes/defaults.inc.php';
+require 'config.php';
+require_once 'includes/definitions.inc.php';
+require 'includes/functions.php';
+require 'html/includes/functions.inc.php';
 
 $console_color = new Console_Color2();
 
@@ -88,7 +90,7 @@
                 $port_speed = humanspeed($port['ifSpeed']);
             }
 
-            if ($port[ifDuplex] != 'unknown') {
+            if ($port['ifDuplex'] != 'unknown') {
                 $port_duplex = $port['ifDuplex'];
             }
 
@@ -108,22 +110,18 @@
         echo $options['list'];
         echo "Usage of console-ui.php:
 
-            -l      What log type we want to see:
-            eventlog = Event log messages
-            syslog = Syslog messages
-
-            -d      Specify the device id to filter results
-
-            --list   What to list
-            devices = list devices and device id's
-
-            --device-stats      Lists the port statistics for a given device
-
-            Examples:
-            #1 php console-ui.php -l eventlog -d 1
-            #2 php console-ui.php --list=devices
-
-            ";
+ -l              What log type we want to see:
+                   eventlog = Event log messages
+                   syslog = Syslog messages
+ -d              Specify the device id to filter results
+ --list          What to list
+                   devices = list devices and device id's
+ --device-stats  Lists the port statistics for a given device
+
+ Examples:
+  #1 php console-ui.php -l eventlog -d 1
+  #2 php console-ui.php --list=devices
+";
         exit;
     }//end if
     echo print $console_color->convert('%rLast update at '.date('Y-m-d h:i:s')."%n\n\n");
diff --git a/scripts/gen_smokeping.php b/scripts/gen_smokeping.php
index b061a4d36875..424a6b8bc701 100755
--- a/scripts/gen_smokeping.php
+++ b/scripts/gen_smokeping.php
@@ -11,9 +11,11 @@
 * the source code distribution for details.
 */
 
-include_once("../includes/defaults.inc.php");
-include_once("../config.php");
-include_once("../includes/definitions.inc.php");
+chdir(realpath(__DIR__ . '/..')); // cwd to the parent directory of this script
+
+include_once("includes/defaults.inc.php");
+include_once("config.php");
+include_once("includes/definitions.inc.php");
 
 ?>
 
diff --git a/scripts/tune_port.php b/scripts/tune_port.php
index e2d6383e298c..410893dfc882 100755
--- a/scripts/tune_port.php
+++ b/scripts/tune_port.php
@@ -1,6 +1,8 @@
 #!/usr/bin/env php
 <?php
 
+chdir(realpath(__DIR__ . '/..')); // cwd to the parent directory of this script
+
 require 'includes/defaults.inc.php';
 require 'config.php';
 require 'includes/definitions.inc.php';
@@ -26,3 +28,5 @@
         rrdtool_tune('port', $rrdfile, $port['ifSpeed']);
     }
 }
+
+rrdtool_close();
diff --git a/scripts/update-sql.php b/scripts/update-sql.php
deleted file mode 100755
index 5f7129eb22ba..000000000000
--- a/scripts/update-sql.php
+++ /dev/null
@@ -1,22 +0,0 @@
-#!/usr/bin/env php
-<?php
-
-// MYSQL Check - FIXME
-// 1 UPDATE
-require 'config.php';
-require 'includes/functions.php';
-
-if ($fd = @fopen($argv[1], 'r')) {
-    $data = fread($fd, 4096);
-    while (!feof($fd)) {
-        $data .= fread($fd, 4096);
-    }
-
-    foreach (explode("\n", $data) as $line) {
-        $update = mysql_query($line);
-        // FIXME check query success?
-        echo "$line \n";
-    }
-} else {
-    echo "ERROR: Could not open file \"$argv[1]\".\n";
-}
diff --git a/snmp-scan.php b/snmp-scan.php
index 095982ed7de0..05bf8c7bbce0 100755
--- a/snmp-scan.php
+++ b/snmp-scan.php
@@ -29,9 +29,9 @@
 use LibreNMS\Exceptions\HostUnreachableException;
 use LibreNMS\Exceptions\HostUnreachablePingException;
 
-$ts = microtime(true);
+chdir(__DIR__); // cwd to the directory containing this script
 
-chdir(dirname($argv[0]));
+$ts = microtime(true);
 
 require 'includes/defaults.inc.php';
 require 'config.php';
diff --git a/snmptrap.php b/snmptrap.php
index 4aaecd21afe7..07bee6bf07dd 100755
--- a/snmptrap.php
+++ b/snmptrap.php
@@ -11,6 +11,8 @@
  * @copyright  (C) 2006 - 2012 Adam Armstrong
  */
 
+chdir(__DIR__); // cwd to the directory containing this script
+
 ini_set('display_errors', 1);
 ini_set('display_startup_errors', 1);
 ini_set('log_errors', 1);
diff --git a/syslog.php b/syslog.php
index 2a2f22484f1c..d8ffd6ba9c1f 100755
--- a/syslog.php
+++ b/syslog.php
@@ -12,6 +12,8 @@
  *
  */
 
+chdir(__DIR__); // cwd to the directory containing this script
+
 require 'includes/defaults.inc.php';
 require 'config.php';
 require 'includes/definitions.inc.php';
diff --git a/validate.php b/validate.php
index c938d8c5438d..50e197dbbf70 100755
--- a/validate.php
+++ b/validate.php
@@ -1,4 +1,4 @@
-#!/usr/bin/php
+#!/usr/bin/env php
 <?php
 
 /*
@@ -13,6 +13,8 @@
  * the source code distribution for details.
  */
 
+chdir(__DIR__); // cwd to the directory containing this script
+
 $options = getopt('m:h::');
 
 if (isset($options['h'])) {
@@ -52,7 +54,7 @@
     $userinfo = posix_getpwuid(posix_geteuid());
     $username = $userinfo['name'];
 } else {
-    $username = getenv('USERNAME') ?: getenv('USER');//http://php.net/manual/en/function.get-current-user.php
+    $username = getenv('USERNAME') ?: getenv('USER'); //http://php.net/manual/en/function.get-current-user.php
 }
 if ($username !== 'root') {
     print_fail("You need to run this script as root");