<?xml version="1.0" encoding="utf-8"?>
<feed xml:lang="en" xmlns="http://www.w3.org/2005/Atom"><title>Recent changes to patches</title><link href="https://sourceforge.net/p/firehol/patches/" rel="alternate"/><link href="https://sourceforge.net/p/firehol/patches/feed.atom" rel="self"/><id>https://sourceforge.net/p/firehol/patches/</id><updated>2015-03-02T07:46:48.662000Z</updated><subtitle>Recent changes to patches</subtitle><entry><title>Use https://github.com/ktsaou/firehol/issues</title><link href="https://sourceforge.net/p/firehol/patches/16/" rel="alternate"/><published>2015-03-02T07:46:48.662000Z</published><updated>2015-03-02T07:46:48.662000Z</updated><author><name>Phil Whineray</name><uri>https://sourceforge.net/u/philwhineray/</uri></author><id>https://sourceforge.net66d1b1fa34bf0c522d50fd341d741f44b9860125</id><summary type="html">&lt;div class="markdown_content"&gt;&lt;p&gt;Use &lt;a href="https://github.com/ktsaou/firehol/issues" rel="nofollow"&gt;https://github.com/ktsaou/firehol/issues&lt;/a&gt; to submit patches&lt;/p&gt;&lt;/div&gt;</summary></entry><entry><title>PATCH: CVS 2013-04-20 get-iana.sh POSIX</title><link href="https://sourceforge.net/p/firehol/patches/15/" rel="alternate"/><published>2013-04-20T09:30:33Z</published><updated>2013-04-20T09:30:33Z</updated><author><name>Jari Aalto</name><uri>https://sourceforge.net/u/jaalto/</uri></author><id>https://sourceforge.netb1f6d257e36468b60f6cffda54d06efff9bc19ba</id><summary type="html">&lt;div class="markdown_content"&gt;&lt;p&gt;This patch is against CVS 2013-04-20 get-iana.sh&lt;/p&gt;
&lt;p&gt;- Use /bin/sh. The file is POSIX compliant now.&lt;br /&gt;
- Remove EOL (end of line) whitespaces.&lt;br /&gt;
- Send errors to STDERR, normal messages to STDOUT.&lt;br /&gt;
- Remove continuation backslashes "\" when shell syntax already&lt;br /&gt;
knowns the line will continue e.g. after pipe "|" character.&lt;br /&gt;
- Fit text in column 80 (comment at the beginning).&lt;br /&gt;
- Instead of ${var}, simplify variables to $var.&lt;br /&gt;
- Prefer POSIX $((i + 1)) instead of bashism $[i + 1]&lt;br /&gt;
- Prefer POSIX $() to archaic backquotes `` [1]&lt;br /&gt;
- Use standard built-in echo(1) instead of external printf(1).&lt;/p&gt;
&lt;p&gt;[1]&lt;br /&gt;
POSIX standard provides the $(...) command substitution syntax, which&lt;br /&gt;
improves legibility and allows nested structures.&lt;br /&gt;
&lt;a href="http://www.opengroup.org/onlinepubs/009695399/utilities/xcu_chap02.html#tag_02_06_03"&gt;http://www.opengroup.org/onlinepubs/009695399/utilities/xcu_chap02.html#tag_02_06_03&lt;/a&gt;&lt;br /&gt;
Also a good read: &lt;a href="http://mywiki.wooledge.org/BashFAQ/082"&gt;http://mywiki.wooledge.org/BashFAQ/082&lt;/a&gt;&lt;/p&gt;&lt;/div&gt;</summary></entry><entry><title>PATCH: CVS 2013-04-20 check-iana.sh POSIX</title><link href="https://sourceforge.net/p/firehol/patches/14/" rel="alternate"/><published>2013-04-20T09:21:44Z</published><updated>2013-04-20T09:21:44Z</updated><author><name>Jari Aalto</name><uri>https://sourceforge.net/u/jaalto/</uri></author><id>https://sourceforge.netae3019714be712943d0f8b9b9cb30d6b30c64322</id><summary type="html">&lt;div class="markdown_content"&gt;&lt;p&gt;This patch is against CVS 2013-04-20 check-iana.sh&lt;/p&gt;
&lt;p&gt;- Use /bin/sh. The file is POSIX compliant now.&lt;br /&gt;
- Instead of ${var}, simplify variabled to $var.&lt;br /&gt;
- Send errors to STDERR, normal messages to STDOUT.&lt;br /&gt;
- Prefer POSIX $() to archaic backquotes `` [1]&lt;br /&gt;
- Prefer simple [ "$var" ] and [ ! "$var" ] test. The -n and -z&lt;br /&gt;
tests are not needed.&lt;/p&gt;&lt;/div&gt;</summary></entry><entry><title>PATCH: CVS 2013-04-20 buildrpm.sh POSIX</title><link href="https://sourceforge.net/p/firehol/patches/13/" rel="alternate"/><published>2013-04-20T09:16:33Z</published><updated>2013-04-20T09:16:33Z</updated><author><name>Jari Aalto</name><uri>https://sourceforge.net/u/jaalto/</uri></author><id>https://sourceforge.net99db7eb8909aa29d10b59bdaf1c1ab7eca53e627</id><summary type="html">&lt;div class="markdown_content"&gt;&lt;p&gt;This patch is against CVS 2013-04-20 buildrpm.sh&lt;/p&gt;
&lt;p&gt;- Use /bin/sh. The file is POSIX compliant now.&lt;br /&gt;
- Instead of ${var}, simplify variabled to $var.&lt;br /&gt;
- Send errors to STDERR, normal messages to STDOUT.&lt;br /&gt;
- Prefer POSIX $() to archaic backquotes `` [1]&lt;br /&gt;
- Prefer simple [ "$var" ] and [ ! "$var" ] test. The -n and -z&lt;br /&gt;
tests are not needed.&lt;br /&gt;
- Use standard built-in echo(1) instead of external printf(1).&lt;br /&gt;
- Prefer built-in "[]" instead of possibly external call test(1).&lt;br /&gt;
- Prefer "[ test ] COND [ test ]" instead of "[ test COND test ]"&lt;br /&gt;
which does not short circuit and may be portability problem [2]&lt;/p&gt;
&lt;p&gt;[1]&lt;br /&gt;
POSIX standard provides the $(...) command substitution syntax, which&lt;br /&gt;
improves legibility and allows nested structures.&lt;br /&gt;
&lt;a href="http://www.opengroup.org/onlinepubs/009695399/utilities/xcu_chap02.html#tag_02_06_03"&gt;http://www.opengroup.org/onlinepubs/009695399/utilities/xcu_chap02.html#tag_02_06_03&lt;/a&gt;&lt;br /&gt;
Also a good read: &lt;a href="http://mywiki.wooledge.org/BashFAQ/082"&gt;http://mywiki.wooledge.org/BashFAQ/082&lt;/a&gt;&lt;/p&gt;
&lt;p&gt;[2] &lt;a href="http://mywiki.wooledge.org/BashPitfalls#A.5B_.22.24foo.22_.3D_bar_.26.26_.22.24bar.22_.3D_foo_.5D"&gt;http://mywiki.wooledge.org/BashPitfalls#A.5B_.22.24foo.22_.3D_bar_.26.26_.22.24bar.22_.3D_foo_.5D&lt;/a&gt;&lt;/p&gt;&lt;/div&gt;</summary></entry><entry><title>PATCH: cvs 2013-04-20 prettyconf.sh POSIX</title><link href="https://sourceforge.net/p/firehol/patches/12/" rel="alternate"/><published>2013-04-20T08:48:01Z</published><updated>2013-04-20T08:48:01Z</updated><author><name>Jari Aalto</name><uri>https://sourceforge.net/u/jaalto/</uri></author><id>https://sourceforge.nete5ac151669729c75d8d0c53edded35b4a67872b3</id><summary type="html">&lt;div class="markdown_content"&gt;&lt;p&gt;This patch is against CVS 2013-04-20, prettyconf.sh&lt;/p&gt;
&lt;p&gt;- Use /bin/sh. The file is POSIX compliant now.&lt;br /&gt;
- Instead of ${var}, simplify variabled to $var.&lt;br /&gt;
- Send errors to STDERR, normal messages to STDOUT. (E.g. missing arg $1)&lt;/p&gt;&lt;/div&gt;</summary></entry><entry><title>Use DENY, not TARPIT on OUTPUT chain</title><link href="https://sourceforge.net/p/firehol/patches/11/" rel="alternate"/><published>2012-03-17T16:17:45Z</published><updated>2012-03-17T16:17:45Z</updated><author><name>Phil Whineray</name><uri>https://sourceforge.net/u/philwhineray/</uri></author><id>https://sourceforge.net5262fd81254dad92fc3adc24d651d060a62a5329</id><summary type="html">&lt;div class="markdown_content"&gt;&lt;p&gt;Seems that TARPIT is only usable on INPUT and FORWARD chains.&lt;/p&gt;
&lt;p&gt;Note, I have not verified this patch beyond checking that it solves the immediate error and there may be a better way to implement it.&lt;/p&gt;
&lt;p&gt;Patch is against 1.273, per the forum report, not latest.&lt;/p&gt;&lt;/div&gt;</summary></entry><entry><title>Parse kernel 3.x correctly</title><link href="https://sourceforge.net/p/firehol/patches/10/" rel="alternate"/><published>2011-10-14T19:57:54Z</published><updated>2011-10-14T19:57:54Z</updated><author><name>Phil Whineray</name><uri>https://sourceforge.net/u/philwhineray/</uri></author><id>https://sourceforge.net33ea07d22cac957ba36ce6ba9e915312c700a6c2</id><summary type="html">&lt;div class="markdown_content"&gt;&lt;p&gt;Was unable to apply patch created by SanskritFritz&lt;/p&gt;
&lt;p&gt;Here is my take on the solution; make the minor number parse correctly for new x.y-extra format as well as old x.y.z-extra.&lt;/p&gt;&lt;/div&gt;</summary></entry><entry><title>Use iptables-restore for faster startup</title><link href="https://sourceforge.net/p/firehol/patches/9/" rel="alternate"/><published>2010-09-27T20:06:21Z</published><updated>2010-09-27T20:06:21Z</updated><author><name>Josh Mahonin</name><uri>https://sourceforge.net/u/jmahonin/</uri></author><id>https://sourceforge.netdb7f6c88074ff40a5496445bf8ec97d40d5a8b9b</id><summary type="html">&lt;div class="markdown_content"&gt;&lt;p&gt;This patches FireHOL to a version we're testing internally on embedded platforms.  On a complex firewall, every call to iptables takes a non-trivial amount of time - on a Soekris board, loading the firewall can be in the range of 30-60s.&lt;/p&gt;
&lt;p&gt;Instead of calling iptables for every rule, instead populate a list of chains, and use iptables-restore to load them all at once.  Every rule is still documented and traceable for debug mode.  Calls to 'touch' have also been replaced with '&amp;gt;', the builtin concat operation which also speeds up processing.&lt;/p&gt;
&lt;p&gt;As well, add ability to match packets based on whether it is marked AND whether it is incoming or outgoing from an interface, via 'inmark' and 'outmark'.&lt;/p&gt;&lt;/div&gt;</summary></entry><entry><title>Experimental IPv6 support</title><link href="https://sourceforge.net/p/firehol/patches/8/" rel="alternate"/><published>2010-05-28T19:23:57Z</published><updated>2010-05-28T19:23:57Z</updated><author><name>Phil Whineray</name><uri>https://sourceforge.net/u/philwhineray/</uri></author><id>https://sourceforge.netf3cf430e9d1046f0b5825c36feb1bf19ad8ed480</id><summary type="html">&lt;div class="markdown_content"&gt;&lt;p&gt;Move some direct command calls to functions, in preparation for adding ipv4/ipv6 logic&lt;/p&gt;&lt;/div&gt;</summary></entry><entry><title>Add further command alternatives for other systems</title><link href="https://sourceforge.net/p/firehol/patches/7/" rel="alternate"/><published>2010-05-23T17:26:31Z</published><updated>2010-05-23T17:26:31Z</updated><author><name>Phil Whineray</name><uri>https://sourceforge.net/u/philwhineray/</uri></author><id>https://sourceforge.net41c7f2ce56e0a3f7040f860deb052ba1c8290541</id><summary type="html">&lt;div class="markdown_content"&gt;&lt;p&gt;Please consider for inclusion:&lt;/p&gt;
&lt;p&gt;With the following additional choices, firehol can be made to run on an openwrt system:&lt;br /&gt;
awk if gawk is not available&lt;br /&gt;
insmod if modprobe is not available&lt;br /&gt;
nothing if renice is not available&lt;/p&gt;&lt;/div&gt;</summary></entry></feed>