summaryrefslogtreecommitdiffstats
path: root/src/etc/inc/priv.inc
diff options
context:
space:
mode:
authorPhil Davis <phil.davis@inf.org>2016-06-28 09:01:31 +0930
committerChris Buechler <cmb@pfsense.org>2016-06-28 22:01:42 -0500
commit6dfb6b27d8320947fb07865ae5e2ebfa17c1e3fc (patch)
tree7d391fe718cb04ec61d0593c2568e87d5ebee1cb /src/etc/inc/priv.inc
parente8019bcde8dc8db6349f93b963d41172a60f06f8 (diff)
downloadpfsense-6dfb6b27d8320947fb07865ae5e2ebfa17c1e3fc.zip
pfsense-6dfb6b27d8320947fb07865ae5e2ebfa17c1e3fc.tar.gz
Update include() to include_once()
For safety, use include_once() when including various "side" files. There are a couple of instances of include("guiconfig,inc") that I changes to use require_once() to be consistent with everywhere else. The remaining cases of include() are just (hundreds of) head.inc and footer.inc - those are only at the start/end of the ordinary top-level interactive PHP files. There is no nested interaction of those to think about, so that should be fine.
Diffstat (limited to 'src/etc/inc/priv.inc')
-rw-r--r--src/etc/inc/priv.inc4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/etc/inc/priv.inc b/src/etc/inc/priv.inc
index 6315f72..b0fcea0 100644
--- a/src/etc/inc/priv.inc
+++ b/src/etc/inc/priv.inc
@@ -88,14 +88,14 @@ function get_priv_files($directory) {
$dir_array = get_priv_files("/etc/inc/priv");
foreach ($dir_array as $file) {
if (!is_dir("/etc/inc/priv/{$file}") && stristr($file, ".inc")) {
- include("/etc/inc/priv/{$file}");
+ include_once("/etc/inc/priv/{$file}");
}
}
if (is_dir("/usr/local/pkg/priv")) {
$dir_array = get_priv_files("/usr/local/pkg/priv");
foreach ($dir_array as $file) {
if (!is_dir("/usr/local/pkg/priv/{$file}") && stristr($file, ".inc")) {
- include("/usr/local/pkg/priv/{$file}");
+ include_once("/usr/local/pkg/priv/{$file}");
}
}
}
OpenPOWER on IntegriCloud