From owner-freebsd-hackers Mon Feb 26 00:26:01 1996 Return-Path: owner-hackers Received: (from root@localhost) by freefall.freebsd.org (8.7.3/8.7.3) id AAA04231 for hackers-outgoing; Mon, 26 Feb 1996 00:26:01 -0800 (PST) Received: from karon.dynas.se (karon.dynas.se [192.71.43.4]) by freefall.freebsd.org (8.7.3/8.7.3) with SMTP id AAA04226 for ; Mon, 26 Feb 1996 00:25:58 -0800 (PST) Received: from host.domain by karon.dynas.se with smtp (Smail3.1.28.1 #5) id m0tqyFh-000EUPC; Mon, 26 Feb 96 09:25:49 +0100 Received: by spirit.dynas.se (Smail3.1.28.1 #32) id m0tqyFh-000JeVC; Mon, 26 Feb 96 09:25:49 +0100 Date: Mon, 26 Feb 1996 09:25:49 +0100 (MET) From: Mikael Hybsch To: freebsd-hackers@freefall.freebsd.org Subject: PATCH: Minor fixes to /usr/sbin/ppp (FreeBSD-stable version) Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-hackers@FreeBSD.ORG Precedence: bulk The current version doesn't allow you to use the "estab" keyword in the filter rules without specifying a source or destination port. The diff for filter.c fixes that. In filter.h, the constant OP_LT should really be 3, otherwise "show ifilter" will index element 4 in the array opnames[0-3]. diff -ur ORG/filter.c ./filter.c --- ORG/filter.c Sun Feb 25 19:07:05 1996 +++ ./filter.c Sun Feb 25 19:47:51 1996 @@ -206,12 +206,6 @@ filterdata.opt.srcop = filterdata.opt.dstop = A_NONE; return(1); } - if (argc < 3) { -#ifdef notdef - printf("bad udp syntax.\n"); -#endif - return(0); - } if (argc >= 3 && STREQ(*argv, "src")) { filterdata.opt.srcop = ParseOp(argv[1]); if (filterdata.opt.srcop == OP_NONE) { diff -ur ORG/filter.h ./filter.h --- ORG/filter.h Sun Feb 25 19:46:48 1996 +++ ./filter.h Sun Feb 25 19:46:58 1996 @@ -48,7 +48,7 @@ #define OP_NONE 0 #define OP_EQ 1 #define OP_GT 2 -#define OP_LT 4 +#define OP_LT 3 struct filterent { int action; /* Filtering action */ -- Mikael Hybsch Email: micke@dynas.se DynaSoft, Dynamic Software AB Phone: +46-8-615 84 00 Hammarby Fabriksv 13, Box 92058 Fax: +46-8-641 92 00 S-120 06 STOCKHOLM, SWEDEN