Bug 29967

Summary: bpf code has a misleading comment
Product: Base System Reporter: Joseph Mallett <jmallett>
Component: kernAssignee: freebsd-bugs (Nobody) <bugs>
Status: Closed FIXED    
Severity: Affects Only Me CC: jmallett
Priority: Normal    
Version: Unspecified   
Hardware: Any   
OS: Any   

Description Joseph Mallett 2001-08-23 00:40:27 UTC
	Ever since some time in ancient history (4.4-lite?), the comment 
for bpfattach has been pretty incorrect when it comes to explaining what 
the parameters are. ifp is not explained, and driverp (which is no longer 
present in FreeBSD) is still explained.

Fix: 

/*
- * Attach an interface to bpf.  driverp is a pointer to a (struct bpf_if 
*)
- * in the driver's softc; dlt is the link layer type; hdrlen is the fixed
- * size of the link header (variable length headers not yet supported).
+ * Attach an interface to bpf. ifp is a pointer to the structure defining
+ * the interface to be attached; dlt is the link layer type; hdrlen is 
the
+ * fixed size of the link header (variable length headers are not
+ * supported, yet.)
  */
 void
 bpfattach(ifp, dlt, hdrlen)--KIUmI7N211ZCGnrMrzc9ziMGlHZpOkFTUzyersa3fA9WkSWJ
Content-Type: text/plain; name="file.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="file.diff"

--- bpf.orig    Wed Aug 22 23:31:16 2001
+++ bpf.c       Wed Aug 22 23:33:27 2001
@@ -1272,9 +1272,10 @@
 }
How-To-Repeat: 	vi /sys/net/bpf.c
	:/bpfattach/
Comment 1 dd freebsd_committer freebsd_triage 2001-08-23 23:38:24 UTC
State Changed
From-To: open->closed

Applied, thanks!