From owner-cvs-sys Tue May 26 04:40:18 1998 Return-Path: Received: (from daemon@localhost) by hub.freebsd.org (8.8.8/8.8.8) id EAA11629 for cvs-sys-outgoing; Tue, 26 May 1998 04:40:18 -0700 (PDT) (envelope-from owner-cvs-sys) Received: from freefall.freebsd.org (freefall.FreeBSD.ORG [204.216.27.21]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id EAA11398; Tue, 26 May 1998 04:39:55 -0700 (PDT) (envelope-from dg@FreeBSD.org) From: David Greenman Received: (from dg@localhost) by freefall.freebsd.org (8.8.8/8.8.5) id EAA10089; Tue, 26 May 1998 04:34:34 -0700 (PDT) Date: Tue, 26 May 1998 04:34:34 -0700 (PDT) Message-Id: <199805261134.EAA10089@freefall.freebsd.org> To: cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG, cvs-sys@FreeBSD.ORG Subject: cvs commit: src/sys/netinet ip_icmp.c Sender: owner-cvs-sys@FreeBSD.ORG X-Loop: FreeBSD.org Precedence: bulk dg 1998/05/26 04:34:33 PDT Modified files: sys/netinet ip_icmp.c Log: Fixed logic in the test to drop ICMP echo and timestamp packets when net.inet.ip.icmp.bmcastecho = 0 by removing the extra check for the address being a multicast address. The test now relies on the link layer flags that indicate it was received via multicast. The previous logic was broken and replied to ICMP echo/timestamp broadcasts even when the sysctl option disallowed them. Reviewed by: wollman Revision Changes Path 1.30 +3 -5 src/sys/netinet/ip_icmp.c