Re: [netfilter-core] [PATCH] compile fix ipfw

[Rusty Russell <rusty@rustcorp.com.au>]

In message <Pine.LNX.4.44.0304162109530.12650-100000@chimarrao.boston.redhat.co
m> you write:
> 
> In the patch that went to marcelo a few days ago the reset
> argument to ip_chain_procinfo() got removed, but there's still
> a code block inside the function that references that variable.
> 
> This patch gets rid of that (presumably old) code block. Note
> that I didn't cc this to Marcelo because I'm not 100% sure, so
> please check it.

This looks fine to me: there used to be magic in 2.0 which meant when
you opened the proc file for writing as well as reading, it'd reset
the counters.

These days, the infrastructure doesn't support such a hack.

Please fwd to Marcelo.

Thanks!
Rusty.

> @@ -1176,12 +1176,6 @@ static int ip_chain_procinfo(int stage, 
>  			len = last_len;
>  			break;
>  		}
> -		else if(reset)
> -		{
> -			/* This needs to be done at this specific place! */
> -			i->fw_pcnt=0L;
> -			i->fw_bcnt=0L;
> -		}
>  		last_len = len;
>  		i=i->fw_next;
>  	}
> 

--
  Anyone who quotes me in their sig is an idiot. -- Rusty Russell.