Index | Thread | Search

From:
Claudio Jeker <cjeker@diehard.n-r-g.com>
Subject:
Re: [PATCH] ripd: fix using rdomain
To:
Alexander Shursha <kekek2@ya.ru>
Cc:
tech@openbsd.org
Date:
Mon, 14 Apr 2025 16:19:23 +0200

Download raw body.

Thread
On Mon, Apr 14, 2025 at 01:17:45PM +0300, Alexander Shursha wrote:
> The rdomain value was set after receiving the routes

Agree, this needs to move up. OK claudio@ (if nobody else wants to jump
in I will commit this tomorrow).

> ---
>  usr.sbin/ripd/kroute.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/usr.sbin/ripd/kroute.c b/usr.sbin/ripd/kroute.c
> index 40aaa06660c..5747c791ae7 100644
> --- a/usr.sbin/ripd/kroute.c
> +++ b/usr.sbin/ripd/kroute.c
> @@ -141,6 +141,7 @@ kr_init(int fs, u_int rdomain, u_int8_t fib_prio)
>  	kr_state.pid = getpid();
>  	kr_state.rtseq = 1;
>  	kr_state.fib_prio = fib_prio;
> +	kr_state.rdomain = rdomain;
>  
>  	RB_INIT(&krt);
>  
> @@ -151,7 +152,6 @@ kr_init(int fs, u_int rdomain, u_int8_t fib_prio)
>  		return (-1);
>  
>  	kr_state.fib_sync = fs; /* now set correct sync mode */
> -	kr_state.rdomain = rdomain;
>  
>  	event_set(&kr_state.ev, kr_state.fd, EV_READ | EV_PERSIST,
>  	    kr_dispatch_msg, NULL);
> -- 
> 2.49.0
> 

-- 
:wq Claudio