Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 13 Aug 2009 09:29:53 +0000 (UTC)
From:      "Bjoern A. Zeeb" <bz@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r196174 - head/sys/net
Message-ID:  <200908130929.n7D9TrGh095296@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: bz
Date: Thu Aug 13 09:29:52 2009
New Revision: 196174
URL: http://svn.freebsd.org/changeset/base/196174

Log:
  Put multiple instructions into a block when iterating; unbreaks
  NET_RT_DUMP, which otherwise only returned information of AF_MAX.
  This was broken in r193232 (save your time - my bug, my fix).
  
  PR:		kern/137700
  Reported by:	Larry Baird (lab gta.com)
  Tested by:	Larry Baird (lab gta.com)
  Reviewed by:	zec, lstewart, qing
  Approved by:	re (kib)

Modified:
  head/sys/net/rtsock.c

Modified: head/sys/net/rtsock.c
==============================================================================
--- head/sys/net/rtsock.c	Thu Aug 13 09:17:07 2009	(r196173)
+++ head/sys/net/rtsock.c	Thu Aug 13 09:29:52 2009	(r196174)
@@ -1473,7 +1473,7 @@ sysctl_rtsock(SYSCTL_HANDLER_ARGS)
 		/*
 		 * take care of routing entries
 		 */
-		for (error = 0; error == 0 && i <= lim; i++)
+		for (error = 0; error == 0 && i <= lim; i++) {
 			rnh = rt_tables_get_rnh(req->td->td_proc->p_fibnum, i);
 			if (rnh != NULL) {
 				RADIX_NODE_HEAD_LOCK(rnh); 
@@ -1482,6 +1482,7 @@ sysctl_rtsock(SYSCTL_HANDLER_ARGS)
 				RADIX_NODE_HEAD_UNLOCK(rnh);
 			} else if (af != 0)
 				error = EAFNOSUPPORT;
+		}
 		break;
 
 	case NET_RT_IFLIST:



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