From owner-cvs-lib Tue Jun 20 11:31:23 1995 Return-Path: cvs-lib-owner Received: (from majordom@localhost) by freefall.cdrom.com (8.6.10/8.6.6) id LAA02745 for cvs-lib-outgoing; Tue, 20 Jun 1995 11:31:23 -0700 Received: (from hsu@localhost) by freefall.cdrom.com (8.6.10/8.6.6) id LAA02734 ; Tue, 20 Jun 1995 11:31:18 -0700 Date: Tue, 20 Jun 1995 11:31:18 -0700 From: Jeffrey Hsu Message-Id: <199506201831.LAA02734@freefall.cdrom.com> To: CVS-commiters, cvs-lib Subject: cvs commit: src/lib/libc/gen closedir.c getcwd.c getttyent.c popen.c Sender: cvs-lib-owner@freebsd.org Precedence: bulk hsu 95/06/20 11:31:17 Modified: lib/libc/gen closedir.c getcwd.c getttyent.c popen.c Log: Don't cast void functions to void. Obtained from: NetBSD commit by jtc on June 16, 1995. From owner-cvs-lib Thu Jun 22 09:56:05 1995 Return-Path: cvs-lib-owner Received: (from majordom@localhost) by freefall.cdrom.com (8.6.10/8.6.6) id JAA16490 for cvs-lib-outgoing; Thu, 22 Jun 1995 09:56:05 -0700 Received: (from ache@localhost) by freefall.cdrom.com (8.6.10/8.6.6) id JAA16480 ; Thu, 22 Jun 1995 09:56:01 -0700 Date: Thu, 22 Jun 1995 09:56:01 -0700 From: "Andrey A. Chernov" Message-Id: <199506221656.JAA16480@freefall.cdrom.com> To: CVS-commiters, cvs-lib Subject: cvs commit: src/lib/libpcap gencode.c Sender: cvs-lib-owner@freebsd.org Precedence: bulk ache 95/06/22 09:56:01 Modified: lib/libpcap gencode.c Log: Fix handling NULL-encapsulated interfaces (lo & tun) Reviewed by: wollman Submitted by: dvv@sprint.net From owner-cvs-lib Fri Jun 23 07:48:00 1995 Return-Path: cvs-lib-owner Received: (from majordom@localhost) by freefall.cdrom.com (8.6.10/8.6.6) id HAA08530 for cvs-lib-outgoing; Fri, 23 Jun 1995 07:48:00 -0700 Received: (from wpaul@localhost) by freefall.cdrom.com (8.6.10/8.6.6) id HAA08520 ; Fri, 23 Jun 1995 07:47:56 -0700 Date: Fri, 23 Jun 1995 07:47:56 -0700 From: Bill Paul Message-Id: <199506231447.HAA08520@freefall.cdrom.com> To: CVS-commiters, cvs-lib Subject: cvs commit: src/lib/libc/gen getnetgrent.c Sender: cvs-lib-owner@freebsd.org Precedence: bulk wpaul 95/06/23 07:47:55 Modified: lib/libc/gen getnetgrent.c Log: Fixes for PR #508 and #509 ('botched 'Bad netgroup' error message' and 'cycle in netgroup check too greedy'). PR #508 is apparently due to an inconsistency in the way the 4.4BSD netgroup code deals with bad netgroups. When 4.4BSD code encounters a badly formed netgroup entry (e.g. (somehost,-somedomain), which, because of the missing comma between the '-' and 'somedomain,' has only 2 fields instead of 3), it generates an error message and then bails out without doing any more processing on the netgroup containing the bad entry. Conversely, every other *NIX in the world that usees netgroups just tries to parse the entry as best it can and then silently continues on its way. The result is that two bad things happen: 1) we ignore other valid entries within the netgroup containing the bogus entry, which prevents us from interoperating with other systems that don't behave this way, and 2) by printing an error to stderr from inside libc, we hose certain programs, in this case rlogind. In the problem report, Bill Fenner noted that the 'B' from 'Bad' was missing, and that rlogind exited immediately after generating the error. The missing 'B' is apparently not caused by any problem in getnetgrent.c; more likely it's getting swallowed up by rlogind somehow, and the error message itself causes rlogind to become confused. I was able to duplicate this problem and discovered that running a simple test program on my FreeBSD system resulted in a properly formatted (if confusing) error, whereas triggering the error by trying to rlogin to the machine yielded the missing 'B' problem. Anyway, the fixes for this are as follows: - The error message has been reformatted so that it prints out more useful information (e.g. Bad entry (somehost,-somedomain) in netgroup "foo"). We check for NULL entries so that we don't print '(null)' anymore too. :) - Rearranged things in parse_netgrp() so that we make a best guess at what bad entries are supposed to look like and then continue processing instead of bailing out. - Even though the error message has been cleaned up, it's wrapped inside a #ifdef DEBUG. This way we match the behavior of other systems. Since we now handle the error condition better anyway, this error message becomes less important. PR #507 is another case of inconsistency. The code that handles duplicate/circular netgroup entries isn't really 'too greedy; -- it's just too noisy. If you have a netgroup containing duplicate entries, the code actually does the right thing, but it also generates an error message. As with the 'Bad netgroup' message, spewing this out from inside libc can also hose certain programs (like rlogind). Again, no other system generates an error message in this case. The only change here is to hide the error message inside an #ifdef DEBUG. Like the other message, it's largely superfluous since the code handles the condition correctly. Note that PR #510 (+@netgroup host matching in /etc/hosts.equiv) is still being investigated. I haven't been able to duplicate it myself, and I strongly suspect it to be a configuration problem of some kind. However, I'm leaving all three PRs open until I get 510 resolved just for the sake of paranoia. From owner-cvs-lib Sat Jun 24 11:53:13 1995 Return-Path: cvs-lib-owner Received: (from majordom@localhost) by freefall.cdrom.com (8.6.10/8.6.6) id LAA21039 for cvs-lib-outgoing; Sat, 24 Jun 1995 11:53:13 -0700 Received: (from joerg@localhost) by freefall.cdrom.com (8.6.10/8.6.6) id LAA21027 ; Sat, 24 Jun 1995 11:53:10 -0700 Date: Sat, 24 Jun 1995 11:53:10 -0700 From: Joerg Wunsch Message-Id: <199506241853.LAA21027@freefall.cdrom.com> To: CVS-commiters, cvs-lib Subject: cvs commit: src/lib/libkvm kvm_getprocs.3 Sender: cvs-lib-owner@freebsd.org Precedence: bulk joerg 95/06/24 11:53:09 Modified: lib/libkvm kvm_getprocs.3 Log: Update the man page for kvm_getprocs.3 to reflect our sysctl-based kvm mechanism. Submitted by: (Vic Abell) From owner-cvs-lib Sat Jun 24 20:35:52 1995 Return-Path: cvs-lib-owner Received: (from majordom@localhost) by freefall.cdrom.com (8.6.10/8.6.6) id UAA06322 for cvs-lib-outgoing; Sat, 24 Jun 1995 20:35:52 -0700 Received: (from bde@localhost) by freefall.cdrom.com (8.6.10/8.6.6) id UAA06312 ; Sat, 24 Jun 1995 20:35:51 -0700 Date: Sat, 24 Jun 1995 20:35:51 -0700 From: Bruce Evans Message-Id: <199506250335.UAA06312@freefall.cdrom.com> To: CVS-commiters, cvs-lib Subject: cvs commit: src/lib/libkvm kvm_getprocs.3 Sender: cvs-lib-owner@freebsd.org Precedence: bulk bde 95/06/24 20:35:50 Modified: lib/libkvm kvm_getprocs.3 Log: 41 headers must be implicitly included and one more () must be explicitly included before kvm_getprocs() can be used.