On 26/10/2020 03:28, Daniel Harding wrote:
When loading ipset files using the loadfile option, skip lines that
start with '#' (disregarding any leading whitespace).

Signed-off-by: Daniel Harding <dhard...@living180.net>

Looks good to me, FWLIW.  Thanks for addressing my suggestion!

---
  ipsets.c | 12 ++++++++++--
  1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/ipsets.c b/ipsets.c
index 280845b..7af795d 100644
--- a/ipsets.c
+++ b/ipsets.c
@@ -16,6 +16,8 @@
   * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
   */
+#include <ctype.h>
+
  #include "ipsets.h"
@@ -337,6 +339,7 @@ load_file(struct fw3_ipset *ipset)
  {
        FILE *f;
        char line[128];
+       char *p;
if (!ipset->loadfile)
                return;
@@ -350,8 +353,13 @@ load_file(struct fw3_ipset *ipset)
                return;
        }
- while (fgets(line, sizeof(line), f))
-               fw3_pr("add %s %s", ipset->name, line);
+       while (fgets(line, sizeof(line), f)) {
+               p = line;
+               while (isblank(*p))
+                       p++;
+               if (*p != '#')
+                       fw3_pr("add %s %s", ipset->name, line);
+       }
fclose(f);
  }


--
Henrique de Moraes Holschuh
www.nic.br

_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to