https://bugzilla.novell.com/show_bug.cgi?id=284998#c4
Lukas Ocilka changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
--- Comment #4 from Lukas Ocilka 2007-07-02 07:18:16 MST ---
A patch:
Index: SuSEFirewallExpertRules.ycp
===================================================================
--- SuSEFirewallExpertRules.ycp (revision 39103)
+++ SuSEFirewallExpertRules.ycp (working copy)
@@ -244,8 +244,17 @@
// Creating new record
string new_rule = params["network"]:"" + "," + params["protocol"]:"";
- if (params["dport"]:"" != "") new_rule = new_rule + "," +
params["dport"]:"";
- if (params["sport"]:"" != "") new_rule = new_rule + "," +
params["sport"]:"";
+
+ // either 'dport' or 'sport'
+ if (params["dport"]:"" != "" || params["sport"]:"" != "") {
+ // 'dport' can be empty even if 'sport' is set
+ new_rule = new_rule + "," + params["dport"]:"";
+
+ if (params["sport"]:"" != "") {
+ new_rule = new_rule + "," + params["sport"]:"";
+ }
+ }
+
if (new_rule == "0/0,all") {
y2warning("Adding rule '%1' that allows everything from all
networks!", new_rule);
}
Fixed in yast2-2.15.38
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.