d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Protect the ifinit() function's internals with splimp() for safety since

Browse Source 
it used to be that way. I'm not sure that it's needed, but it does
walk the ifp list..

Incidently, there's nothing to sanity check the ifq_maxlen on loaded
interfaces..
This commit is contained in:
Peter Wemm 1999-04-26 09:02:40 +00:00
parent 61a6c2e81d
commit 8ba5bdae2a
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
sys/net/if.c
View File

@ -31,7 +31,7 @@
* SUCH DAMAGE.
*
* @(#)if.c 8.3 (Berkeley) 1/4/94
* $Id: if.c,v 1.66 1999/02/19 13:41:35 phk Exp $
* $Id: if.c,v 1.67 1999/04/16 21:22:44 peter Exp $
*/
#include "opt_compat.h"
@ -76,22 +76,23 @@ struct ifnethead ifnet; /* depend on static init XXX */
*
* Routines with ifa_ifwith* names take sockaddr *'s as
* parameters.
*
* This routine assumes that it will be called at splimp() or higher.
*/
/* ARGSUSED*/
void
ifinit(dummy)
void *dummy;
{
register struct ifnet *ifp;
struct ifnet *ifp;
int s;
s = splimp();
for (ifp = ifnet.tqh_first; ifp; ifp = ifp->if_link.tqe_next)
if (ifp->if_snd.ifq_maxlen == 0) {
printf("%s%d XXX: driver didn't set ifq_maxlen\n",
ifp->if_name, ifp->if_unit);
ifp->if_snd.ifq_maxlen = ifqmaxlen;
}
splx(s);
if_slowtimo(0);
}