Skip to content

Commit

Permalink
firewall: remove arbitrary restriction; pf(4) can cope with it
Browse files Browse the repository at this point in the history
  • Loading branch information
fichtner committed Apr 1, 2015
1 parent ec89f5e commit ad5065e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
5 changes: 1 addition & 4 deletions src/www/firewall_rules_edit.php
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
<?php

/*
Copyright (C) 2014 Deciso B.V.
Copyright (C) 2005 Scott Ullrich (sullrich@gmail.com)
Expand Down Expand Up @@ -283,10 +284,6 @@ function is_aoadv_used($rule_config) {
}
}

if (($_POST['proto'] != "tcp") && ($_POST['proto'] != "udp") && ($_POST['proto'] != "tcp/udp") && ($_POST['proto'] != "icmp")) {
if($_POST['ipprotocol'] == "inet46")
$input_errors[] = gettext("You can not assign a protocol other then ICMP, TCP, UDP or TCP/UDP to a rule that applies to IPv4 and IPv6");
}
if (($_POST['proto'] == "icmp") && ($_POST['icmptype'] <> "")){
if($_POST['ipprotocol'] == "inet46")
$input_errors[] = gettext("You can not assign a ICMP type to a rule that applies to IPv4 and IPv6");
Expand Down
3 changes: 2 additions & 1 deletion src/www/services_snmp.php
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
<?php

/*
Copyright (C) 2014-2015 Deciso B.V.
Copyright (C) 2003-2004 Manuel Kasper <mk@neon1.net>.
Expand Down Expand Up @@ -342,7 +343,7 @@ function enable_change(whichone) {
<td width="22%" valign="top" class="vncellreq">Write community string</td>
<td width="78%" class="vtable">
<input name="rwcommunity" type="text" class="formfld unknown" id="rwcommunity" size="40" value="<?=htmlspecialchars($pconfig['rwcommunity']);?>" />
<br />Please use something other then &quot;private&quot; here<br />
<br />Please use something other than &quot;private&quot; here<br />
</td>
</tr>
*/
Expand Down

0 comments on commit ad5065e

Please sign in to comment.