diff options
| author | Phil Davis <phil.davis@inf.org> | 2016-06-28 09:01:31 +0930 |
|---|---|---|
| committer | Chris Buechler <cmb@pfsense.org> | 2016-06-28 22:01:42 -0500 |
| commit | 6dfb6b27d8320947fb07865ae5e2ebfa17c1e3fc (patch) | |
| tree | 7d391fe718cb04ec61d0593c2568e87d5ebee1cb /src/etc/inc/captiveportal.inc | |
| parent | e8019bcde8dc8db6349f93b963d41172a60f06f8 (diff) | |
| download | pfsense-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/captiveportal.inc')
| -rw-r--r-- | src/etc/inc/captiveportal.inc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/etc/inc/captiveportal.inc b/src/etc/inc/captiveportal.inc index dfa9e3c..e2f6a51 100644 --- a/src/etc/inc/captiveportal.inc +++ b/src/etc/inc/captiveportal.inc @@ -2289,7 +2289,7 @@ function portal_allow($clientip, $clientmac, $username, $password = null, $attri $message = 0; } - include("{$g['varetc_path']}/captiveportal-{$cpzone}-logout.html"); + include_once("{$g['varetc_path']}/captiveportal-{$cpzone}-logout.html"); } else { portal_reply_page($my_redirurl, "redir", "Just redirect the user."); |
