netdev
[Top] [All Lists]

Re: [IPv4]: More fib_alias insertion fixes

To: ja@xxxxxx (Julian Anastasov)
Subject: Re: [IPv4]: More fib_alias insertion fixes
From: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Date: Mon, 27 Sep 2004 07:23:16 +1000
Cc: herbert@xxxxxxxxxxxxxxxxxxx, davem@xxxxxxxxxxxxx, netdev@xxxxxxxxxxx
In-reply-to: <Pine.LNX.4.58.0409261611230.8973@xxxxxxxxxxxx>
Organization: Core
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: tin/1.7.4-20040225 ("Benbecula") (UNIX) (Linux/2.4.27-hx-1-686-smp (i686))
Julian Anastasov <ja@xxxxxx> wrote:
> 
>        True, but in your patch2 you return fa after
> list_for_each_entry which is not valid when the loop terminates.
> Just change it to return the fa into the loop.

You're right, we dereference it in fn_hash_insert.

> [-- text/plain, encoding base64, charset: US-ASCII, 28 lines, name: 
> fib_ins3.diff --]
> [-- Description: fib_alias ordered by TOS --]
> 
> diff -ur v2.6.9-rc2-bk10/linux/net/ipv4/fib_hash.c linux/net/ipv4/fib_hash.c
> --- v2.6.9-rc2-bk10/linux/net/ipv4/fib_hash.c   2004-09-26 16:52:11.000000000 
> +0300

Looks good.

Thanks,
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

<Prev in Thread] Current Thread [Next in Thread>