[PATCH firewall3 v2] ipsets: allow commented lines with loadfile

Henrique de Moraes Holschuh henrique at nic.br
Mon Oct 26 08:53:22 EDT 2020


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 <dharding at 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



More information about the openwrt-devel mailing list