| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
| |
Because it contains $xml which is itself wrapped in htmlspecialchars() for display.
|
|
|
|
|
|
|
|
|
|
|
| |
I had a system that had upgraded but the Notes package had not managed to (re)install. In that situation the Notes item is on the Status menu but there is no notes.xml or other notes code on the the system.
When I go to Status->Notes I get:
Warning: substr_compare(): The start position cannot exceed initial string length in /usr/local/www/pkg.php on line 56 File not found notes.xml
The warning is because realpath() returns false in this case and cannot be used is a parameter to substr_compare().
Handle this case, and make the error message more informative.
Note: In RELENG_2_2 print_info_box_np() puts a "Close" button and has no way to stop that. It looks dumb, but that can be cleaned up in 2.3 master. I am just doing this in RELENG_2_2 to avoid the PHP warning.
|
|
|
|
| |
If you enter invalid stuff in the interface description - e.g. "123" - and press save, then you get a warning about it, but the breadcrumb changes to "Interfaces: 123" - the wrongly entered description (that was not applied).
If you enter a valid string for 'descr' then by this point $wancfg has the new value anyway and so the breadcrumb will change correctly if you make a valid entry in 'descr' and save.
|
|
|
|
| |
- RELENG_2_2
|
|
|
|
|
|
|
|
|
|
| |
This has annoyed me in the past. I noticed it in 2.3-BETA just now and went to look on a 2.2.6 system and realized it is an issue there also.
1) When you click to add a new limiter or new queue, then the "Add new queue" button is displayed down the bottom. That is dumb - you are already adding and actually you need to enter the data and save first before adding something else.
2) When you are adding something (like in (1)) the Delete button is displayed. That is also dumb - there is nothing to delete until you have press save to add it!
3) When clicking on a Limiter at the top of the tree, the Delete button syas "Delete this queue", but it is the top level limiter that is being edited and could be deleted, The $_GET/$_POST that happens sends "pipe" and "queue" both with the name of the top-level limiter. So in the end $queue always exists. So we only want the text "Delete this queue" if $queue exists and the queue name is NOT the same as the pipe name.
I am submitting this for RELENG_2_2 because I wanted to sort that out anyway to understand what was wrong and how it should behave.
Then I can look at 2.3-BETA and make it work nicely there...
|
|
|
| |
Might as well provide the fixes for RELENG_2_2 since it is easy and that is where I first tested. Even if there is never a 2.2.7, at least the fixes are there in GitHub to be seen.
|
|
|
|
| |
for how to handle this properly now while respecting CSRF.
|
|
|
|
| |
be scripted. Other actions need its protection.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
disable on inactive interfaces"
This reverts commit 6310eaa2c75424da6a322a956bbf228b36222918.
|
| |
|
|
|
|
| |
and pbi_info.
|
|\ |
|
| |
| |
| | |
This is code for RELENG_2_2 pfSense 2.2.5 that will limit the number of rows in the alias info popup to 100.
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Use a different flag file to indicate that a package reinstall is
required after a reboot is done first. This avoids the possibility that
the user navigates in the webGUI during the time while the shutdown is
in progress and is accidentally presented with the reinstall all
packages GUI button.
Early in rc.bootup switch the flag file to use its ordinary name, so
that all subsequent code in boot scripts and webGUI will work as it
already does to handle the package reinstall and notifying the user that
a package reinstall is about to be done or in progress...
|
| |
| |
| |
| |
| |
| | |
ability to pass relative paths
Restricts them to files under their intended base directories.
|
| |
| |
| |
| | |
never ported over from the package (and is messy to do so). Ticket #5545
|
|\ \ |
|
| | |
| | |
| | |
| | | |
There is a variable cut-paste error here. Actually it never makes any difference in practice because outgoing_num_tcp and incoming_num_tcp are both set, so the logic comes out the same.
I noticed this while reviewing stuff in master for 2.3-ALPHA - might as well back-port stuff like this so that nobody who notices will worry about it in RELENG_2_2.
|
|\ \ \ |
|
| | |/
| |/|
| | |
| | | |
The block private networks and anti-lockout rules display in the firewall rules display with the schedule column blank. But the block bogon networks has an asterisk. That looks odd. I noticed it just now when looking at firewall rules in 2.3.
Might as well fix it here in RELENG_2_2.
|
|\ \ \ |
|
| |/ / |
|
|/ /
| |
| |
| | |
The code in RELENG_2_2 for system_usermanager_settings.php seems to work OK in RELENG_2_2 so I did not touch anything there.
But diag_authentication.php had nothing about $pconfig on initial page load. This extra code makes it select a reasonable default auth server when the page first loads. After that, each time the user does "Test", the page comes back with the previously selected auth server, user name and password (like it did already) - so that seems good.
|
|/
|
|
| |
Fix ID used by child SA disconnect button.
|
| |
|
|
|
|
| |
committed with a different name
|
| |
|
|
|
|
| |
should be an isset on the remote SPI field.
|
|
|
|
| |
the original m0n0wall BSD 2 clause.
|
|
|
|
| |
VICI, also use local-nat-t and remote-nat-t to detect when NAT-T is being used
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|\ |
|
| |
| |
| |
| | |
for RELENG_2_2
|
|\ \ |
|
| |/
| |
| | |
and forum https://forum.pfsense.org/index.php?topic=101276.0
|
|/
|
|
| |
complications of 0 due to PHP stupidity. Upgrade config to add 1 to any configured log levels. Default to 1 as log level where none is configured by the user. Ticket #5340
|
|
|
|
|
| |
This was fixed in master for 2.3 by https://github.com/pfsense/pfsense/commit/50e6c063e6ec148917ff0bcb0bce8b0a08df5792 - in master all of these $tab_array entries, in each file that they appear in, had been modified to just use the $tab_array[] = form.
But in RELENG_2_2 that has not happened. So it seems nicer to just fix the numbering here to match what is already in the other interfaces_*.php files in RELENG_2_2.
Note that the code works OK without this "fix" - display_top_tabs() just loops through the existing array keys anyhow and so did not notice the missing number.
|