Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 9 Jun 1995 17:58:15 +0200
From:      Stefan Petri <petri@ibr.cs.tu-bs.de>
To:        FreeBSD-gnats-submit@FreeBSD.org, bugs@FreeBSD.org
Subject:   missing break in isa/if_el.c causes panic in bpf.c + fix
Message-ID:  <199506091558.RAA05474@achill.ibr.cs.tu-bs.de>

next in thread | raw e-mail | index | archive | help

>Submitter-Id:   net
>Originator:     Stefan Petri
>Organization:   TU Braunschweig, Inst. f. Betriebssysteme u. Rechnerverbund
>Confidential:   no
>Synopsis:       missing break in isa/if_el.c causes panic in bpf.c + fix
>Severity:       serious
>Priority:       low
>Category:       i386
>Release:        FreeBSD 2.0-950412-SNAP i386
>Class:          sw-bug
>Environment: 

	FreeBSD 2.0-950412-SNAP, some PC with (dont laugh) 3C501 Ethernet
	card.

>Description: 

	The kernel's bpf module panics when turning off promiscous mode
	on a 3C501 ethernet card (interface el0). The reason is a missing
	``break'' statement after the SIOCSIFFLAGS case in the big switch
	in el_ioctl() in i386/isa/if_el.c

>How-To-Repeat: 

	> tcpdump -i el0
	tcpdump: listening on el0
	[..packet printout deleted..]
	^C

	panic: bpf: ifpromisc failed
	[..]

>Fix: 
	

*** /usr/src/sys/i386/isa/if_el.c-2.0-950412-SNAP	Wed Apr 12 22:47:48 1995
--- if_el.c	Thu Jun  8 19:13:53 1995
***************
*** 768,773 ****
--- 768,774 ----
  		    	    ((ifp->if_flags & IFF_RUNNING) == 0))
  				el_init(ifp->if_unit);
  		}
+ 		break;
  
  	case SIOCSIFMTU:
  





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