Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 26 Apr 2014 17:16:18 -0600
From:      Ian Lepore <ian@FreeBSD.org>
To:        "Alexander V. Chernikov" <melifaro@FreeBSD.org>
Cc:        svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: svn commit: r264987 - head/sys/net
Message-ID:  <1398554178.61646.149.camel@revolution.hippie.lan>
In-Reply-To: <201404262232.s3QMW4pe085571@svn.freebsd.org>
References:  <201404262232.s3QMW4pe085571@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 2014-04-26 at 22:32 +0000, Alexander V. Chernikov wrote:
> Author: melifaro
> Date: Sat Apr 26 22:32:04 2014
> New Revision: 264987
> URL: http://svnweb.freebsd.org/changeset/base/264987
> 
> Log:
>   Determine fibnum once in the beginning of route_output().
>   
>   MFC after:	1 month
> 
> Modified:
>   head/sys/net/rtsock.c

> Modified: head/sys/net/rtsock.c
> ==============================================================================
> --- head/sys/net/rtsock.c	Sat Apr 26 21:03:41 2014	(r264986)
> +++ head/sys/net/rtsock.c	Sat Apr 26 22:32:04 2014	(r264987)
> @@ -526,7 +526,7 @@ route_output(struct mbuf *m, struct sock
>  	struct sockaddr_in6 *sin6;
>  	int i, rti_need_deembed = 0;
>  #endif
> -	int len, error = 0;
> +	int len, error = 0, fibnum;
>  	struct ifnet *ifp = NULL;
>  	union sockaddr_union saun;
>  	sa_family_t saf = AF_UNSPEC;
> @@ -582,6 +582,8 @@ route_output(struct mbuf *m, struct sock
>  			senderr(error);
>  	}
>  
> +	fibnum = so->so_fibnum;
> +
>  	/*
>  	 * The given gateway address may be an interface address.
>  	 * For example, issuing a "route change" command on a route
> @@ -596,7 +598,7 @@ route_output(struct mbuf *m, struct sock
>  
>  		bzero(&gw_ro, sizeof(gw_ro));
>  		gw_ro.ro_dst = *info.rti_info[RTAX_GATEWAY];
> -		rtalloc_ign_fib(&gw_ro, 0, so->so_fibnum);
> +		rtalloc_ign_fib(&gw_ro, 0, fibnum);
>  		/* 
>  		 * A host route through the loopback interface is 
>  		 * installed for each interface adddress. In pre 8.0
> @@ -637,7 +639,7 @@ route_output(struct mbuf *m, struct sock
>  			break;
>  		}
>  		error = rtrequest1_fib(rtm->rtm_type, &info, &saved_nrt,
> -		    so->so_fibnum);
> +		    fibnum);
>  		if (error == 0 && saved_nrt != NULL) {
>  #ifdef INET6
>  			rti_need_deembed = (V_deembed_scopeid) ? 1 : 0;
> @@ -663,8 +665,7 @@ route_output(struct mbuf *m, struct sock
>  #endif
>  			break;
>  		}
> -		error = rtrequest1_fib(RTM_DELETE, &info, &saved_nrt,
> -		    so->so_fibnum);
> +		error = rtrequest1_fib(RTM_DELETE, &info, &saved_nrt, fibnum);
>  		if (error == 0) {
>  			RT_LOCK(saved_nrt);
>  			rt = saved_nrt;
> @@ -677,8 +678,7 @@ route_output(struct mbuf *m, struct sock
>  		break;
>  
>  	case RTM_GET:
> -		rnh = rt_tables_get_rnh(so->so_fibnum,
> -		    info.rti_info[RTAX_DST]->sa_family);
> +		rnh = rt_tables_get_rnh(fibnum, saf);
>  		if (rnh == NULL)
>  			senderr(EAFNOSUPPORT);
>  
> @@ -867,7 +867,7 @@ flush:
>  			m_adj(m, rtm->rtm_msglen - m->m_pkthdr.len);
>  	}
>  	if (m) {
> -		M_SETFIB(m, so->so_fibnum);
> +		M_SETFIB(m, fibnum);
>  		m->m_flags |= RTS_FILTER_FIB;
>  		if (rp) {
>  			/*
> 

I've got build breakage that I think may be from this change...

/local/build/staging/freebsd/head/src/sys/net/rtsock.c: In function 'route_output':
/local/build/staging/freebsd/head/src/sys/net/rtsock.c:529: warning: 'fibnum' may be used uninitialized in this function

-- Ian





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1398554178.61646.149.camel>