From: Kevin Day <kevin@kevux.org>
Date: Sun, 10 Jun 2012 18:31:20 +0000 (-0500)
Subject: Bugfix: Fix usage of rule_contents for protocol none option
X-Git-Tag: 0.3.0~6
X-Git-Url: https://git.kevux.org/?a=commitdiff_plain;h=9632ef57518d1209c39ea8bd72ee0b03c7ea7797;p=fll

Bugfix: Fix usage of rule_contents for protocol none option

The if condition that is checking for the existing of the protocol option 'none' was improperly setup.
It was using the rule_objects instead of the rule_contents and therefore would never validate.
---

diff --git a/level_3/firewall/c/private-firewall.c b/level_3/firewall/c/private-firewall.c
index d142ef5..be151d6 100644
--- a/level_3/firewall/c/private-firewall.c
+++ b/level_3/firewall/c/private-firewall.c
@@ -222,7 +222,7 @@
 
           if (f_error_is_error(status)) break;
 
-          if (fl_compare_strings(local.buffer.string + local.rule_objects.array[i].start, (f_string) firewall_protocol_none, length, firewall_protocol_none_length) == f_equal_to) {
+          if (fl_compare_strings(local.buffer.string + local.rule_contents.array[i].array[0].start, (f_string) firewall_protocol_none, length, firewall_protocol_none_length) == f_equal_to) {
             use_protocol = f_false;
           } else {
             strncat(protocol.string, local.buffer.string + local.rule_contents.array[i].array[0].start, length);