netdev - Re: [PATCH 6/42] decnet: Fix set-but-unused variable.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1303092729.5282.1005.camel@localhost>
Date:	Mon, 18 Apr 2011 03:12:09 +0100
From:	Ben Hutchings <bhutchings@...arflare.com>
To:	David Miller <davem@...emloft.net>
Cc:	netdev@...r.kernel.org
Subject: Re: [PATCH 6/42] decnet: Fix set-but-unused variable.

On Sun, 2011-04-17 at 17:32 -0700, David Miller wrote:
> "next" in dn_rebuild_zone() is set but not actually used,
> kill it off.
> 
> Signed-off-by: David S. Miller <davem@...emloft.net>
> ---
>  net/decnet/dn_table.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
> 
> diff --git a/net/decnet/dn_table.c b/net/decnet/dn_table.c
> index 99d8d3a..d8ea583 100644
> --- a/net/decnet/dn_table.c
> +++ b/net/decnet/dn_table.c
> @@ -124,11 +124,10 @@ static inline void dn_rebuild_zone(struct dn_zone *dz,
>  				   int old_divisor)
>  {
>  	int i;
> -	struct dn_fib_node *f, **fp, *next;
> +	struct dn_fib_node *f, **fp;
>  
>  	for(i = 0; i < old_divisor; i++) {
>  		for(f = old_ht[i]; f; f = f->fn_next) {
> -			next = f->fn_next;
>  			for(fp = dn_chain_p(f->fn_key, dz);
>  				*fp && dn_key_leq((*fp)->fn_key, f->fn_key);
>  				fp = &(*fp)->fn_next)

This function is rebuilding a hash table after the number of buckets is
changed.  After moving each element into a new bucket, it needs to carry
on iterating over the old bucket.  Therefore the 'next' variable is
really needed and the second for-loop should use it: 'f = next', not
'f = f->fn_next'.

Currently this must just leak routes as the table grows, but I suppose
no-one really uses DECnet any more.

Ben.

-- 
Ben Hutchings, Senior Software Engineer, Solarflare
Not speaking for my employer; that's the marketing department's job.
They asked us to note that Solarflare product names are trademarked.

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html