From 93dcedc1ec731d016e1f87fd8a77284f38c48d0b Mon Sep 17 00:00:00 2001 From: N0YB Date: Mon, 27 Jan 2014 22:49:32 -0800 Subject: XHTML Compliance - Status: System logs: Firewall An attribute value specification must be an attribute value literal unless SHORTTAG YES is specified Quote (or escape) the quotes so they show up in the HTML. Use style where attribute not supported. Relocate tfoot to supported location and add tbody. Apply htmlspecialchars to rule name. Assorted tweaks. --- etc/inc/filter_log.inc | 2 +- usr/local/www/diag_logs_filter.php | 161 +++++++++++++++++++--------------- usr/local/www/javascript/sorttable.js | 8 +- 3 files changed, 94 insertions(+), 77 deletions(-) diff --git a/etc/inc/filter_log.inc b/etc/inc/filter_log.inc index 31d3f60..c0067fa 100644 --- a/etc/inc/filter_log.inc +++ b/etc/inc/filter_log.inc @@ -294,7 +294,7 @@ function find_rule_by_number_buffer($rulenum, $type){ } else { $ruleString = $buffer_rules_normal["@".$rulenum]; list(,$rulename,) = explode("\"",$ruleString); - $rulename = str_replace("USER_RULE: ",'USER_RULE ',$rulename); + $rulename = str_replace("USER_RULE: ",'USER_RULE ',htmlspecialchars($rulename)); } return $rulename." (@".$rulenum.")"; } diff --git a/usr/local/www/diag_logs_filter.php b/usr/local/www/diag_logs_filter.php index 6338396..4192649 100755 --- a/usr/local/www/diag_logs_filter.php +++ b/usr/local/www/diag_logs_filter.php @@ -68,7 +68,7 @@ $filtersubmit = getGETPOSTsettingvalue('filtersubmit', null); if ($filtersubmit) { $interfacefilter = getGETPOSTsettingvalue('interface', null); $filtertext = getGETPOSTsettingvalue('filtertext', ""); - $filterlogentries_qty = getGETPOSTsettingvalue('filterlogentries_qty', null); + $filter_qty = getGETPOSTsettingvalue('filter_qty', null); } $filterlogentries_submit = getGETPOSTsettingvalue('filterlogentries_submit', null); @@ -100,6 +100,9 @@ $nentries = $config['syslog']['nentries']; if ($filterlogentries_qty) $nentries = $filterlogentries_qty; +if ($filter_qty) + $nentries = $filter_qty; + if (!$nentries) $nentries = 50; @@ -135,11 +138,11 @@ include("head.inc");
- "> +
"> - + + - - + + + + + + + + - "); - + $int = strtolower($filterent['interface']); $proto = strtolower($filterent['proto']); if(is_ipaddrv6($filterent['srcip'])) { @@ -348,7 +361,7 @@ include("head.inc"); $dststr = $filterent['dstip'] . get_port_with_service($filterent['dstport'], $proto); ?> + + + + + + + + - - - - -
-
> +
>
- - - - - - -
-
-
- /> Pass
- /> Block
- /> Reject
+
+
+
+ /> Pass
+ /> Block
+ /> Reject
 -
-
+
+
-
-
+
+
-
-
+
+
-
-
+
+
-
-
+
+
-
-
+ 		+
+
-
-
+ 		+
+
-
-
+ 		+
+
-
-
+ 		+
+
-
" />
+ 		+
" />
- ', '');?>   -    + + ', '');?>   +   
-
> +
>
-
-
-
- + $ifacename): ?> - +
-
-
+
+
-
-
+
+
-
 
-
" />
+
 
+
" />
- ', '');?>   + + ', '');?>  
-
+
| |
-
+
Show/hide rule descriptions @@ -300,17 +303,27 @@ include("head.inc");
- - + + - - +
+
+ " /> +
+
+
', outputrule);"> <?php echo $filterent['act'];?> @@ -333,7 +346,7 @@ include("head.inc"); ".find_rule_by_number_buffer($filterent['rulenum'],$filterent['act'])."
 - &dialog_output=true', outputrule);" title=""> + &dialog_output=true', outputrule);" title=""> Icon Reverse Resolve with DNS "> Icon Reverse Resolve with DNS @@ -357,7 +370,7 @@ include("head.inc"); - &dialog_output=true', outputrule);" title=""> + &dialog_output=true', outputrule);" title=""> Icon Reverse Resolve with DNS "> Icon Reverse Resolve with DNS @@ -379,27 +392,31 @@ include("head.inc"); +
+
+ " /> +
+
-
- " /> -
-
diff --git a/usr/local/www/javascript/sorttable.js b/usr/local/www/javascript/sorttable.js index ce2e68a..a780aa5 100644 --- a/usr/local/www/javascript/sorttable.js +++ b/usr/local/www/javascript/sorttable.js @@ -74,8 +74,8 @@ sorttable = { if (headrow == undefined) return; - if (table.getAttribute("sortableMultirow") != undefined) - sortableMultirow = parseInt(table.getAttribute("sortableMultirow")); + if (table.style.sortableMultirow != undefined) + sortableMultirow = parseInt(table.style.sortableMultirow); else sortableMultirow = 1; @@ -247,8 +247,8 @@ sorttable = { hasInputs = (typeof node.getElementsByTagName == 'function') && node.getElementsByTagName('input').length; - if (node.getAttribute("sorttable_customkey") != null) { - return node.getAttribute("sorttable_customkey"); + if (node.style.sorttable_customkey != null) { + return node.style.sorttable_customkey; } else if (typeof node.textContent != 'undefined' && !hasInputs) { return node.textContent.replace(/^\s+|\s+$/g, ''); -- cgit v1.1