From jmallett@mail.newgold.net  Wed Aug 22 16:37:46 2001
Return-Path: <jmallett@mail.newgold.net>
Received: from mail.newgold.net (aphex.newgold.net [209.42.222.44])
	by hub.freebsd.org (Postfix) with SMTP id 5C23B37B405
	for <FreeBSD-gnats-submit@freebsd.org>; Wed, 22 Aug 2001 16:37:45 -0700 (PDT)
	(envelope-from jmallett@mail.newgold.net)
Received: (qmail 37869 invoked by uid 1000); 22 Aug 2001 23:37:37 -0000
Message-Id: <20010822233737.37868.qmail@mail.newgold.net>
Date: 22 Aug 2001 23:37:37 -0000
From: Joseph Mallett <jmallett@xMach.org>
To: FreeBSD-gnats-submit@freebsd.org
Cc: Joseph Mallett <jmallett@xMach.org>
Subject: src/sys/net/bpf.c comments are incorrect
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         29967
>Category:       kern
>Synopsis:       bpf code has a misleading comment
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    freebsd-bugs
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          update
>Submitter-Id:   current-users
>Arrival-Date:   Wed Aug 22 16:40:27 PDT 2001
>Closed-Date:    Thu Aug 23 15:38:24 PDT 2001
>Last-Modified:  Thu Aug 23 15:38:39 PDT 2001
>Originator:     Joseph Mallett
>Release:        FreeBSD 4.4-RC i386
>Organization:
xMach
>Environment:
System: FreeBSD Aphex.NewGold.NET 4.4-RC FreeBSD 4.4-RC #0: Mon Aug 20 06:41:21 GMT 2001 jmallett@Aphex.NewGold.NET:/usr/people/jmallett/obj i386


	
>Description:
	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.

>How-To-Repeat:
	vi /sys/net/bpf.c
	:/bpfattach/
>Fix:
--- bpf.orig    Wed Aug 22 23:31:16 2001
+++ bpf.c       Wed Aug 22 23:33:27 2001
@@ -1272,9 +1272,10 @@
 }

 /*
- * 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)
>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->closed 
State-Changed-By: dd 
State-Changed-When: Thu Aug 23 15:38:24 PDT 2001 
State-Changed-Why:  
Applied, thanks! 

http://www.FreeBSD.org/cgi/query-pr.cgi?pr=29967 
>Unformatted:
