From owner-cvs-src-old@FreeBSD.ORG Thu Sep 2 17:44:01 2010 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 141651065909 for ; Thu, 2 Sep 2010 17:44:01 +0000 (UTC) (envelope-from bz@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id DB75B8FC13 for ; Thu, 2 Sep 2010 17:44:00 +0000 (UTC) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.4/8.14.4) with ESMTP id o82Hi07E018469 for ; Thu, 2 Sep 2010 17:44:00 GMT (envelope-from bz@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.4/8.14.4/Submit) id o82Hi09Y018468 for cvs-src-old@freebsd.org; Thu, 2 Sep 2010 17:44:00 GMT (envelope-from bz@repoman.freebsd.org) Message-Id: <201009021744.o82Hi09Y018468@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to bz@repoman.freebsd.org using -f From: "Bjoern A. Zeeb" Date: Thu, 2 Sep 2010 17:43:44 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src/sys/netinet ip_input.c ip_var.h src/sys/netinet6 ip6_input.c ip6_var.h X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 02 Sep 2010 17:44:01 -0000 bz 2010-09-02 17:43:44 UTC FreeBSD src repository Modified files: sys/netinet ip_input.c ip_var.h sys/netinet6 ip6_input.c ip6_var.h Log: SVN rev 212155 on 2010-09-02 17:43:44Z by bz MFp4 CH=183052 183053 183258: In protosw we define pr_protocol as short, while on the wire it is an uint8_t. That way we can have "internal" protocols like DIVERT, SEND or gaps for modules (PROTO_SPACER). Switch ipproto_{un,}register to accept a short protocol number(*) and do an upfront check for valid boundries. With this we also consistently report EPROTONOSUPPORT for out of bounds protocols, as we did for proto == 0. This allows a caller to not error for this case, which is especially important if we want to automatically call these from domain handling. (*) the functions have been without any in-tree consumer since the initial introducation, so this is considered save. Implement ip6proto_{un,}register() similarly to their legacy IP counter parts to allow modules to hook up dynamically. Reviewed by: philip, will MFC after: 1 week Revision Changes Path 1.386 +6 -10 src/sys/netinet/ip_input.c 1.120 +2 -2 src/sys/netinet/ip_var.h 1.144 +58 -0 src/sys/netinet6/ip6_input.c 1.58 +3 -0 src/sys/netinet6/ip6_var.h