Small touchups:

- Fix typos in comments in hash.c.
- Remove unneeded and unused member from grouplist struct in hash.h.
  (Curiously, the compiler never complained about this even though the
   member was of type 'struct grps' which is not defined anywhere in
   this program.)
- char ch -> int ch in revnetgroup.c.
- char *argv[0]; -> char *argv[]; also in revnetgroup.c.
- Force the user to specify at least one of the -u or -h flags
  and complain if they specify both.

libexec/revnetgroup/hash.c

@@ -29,7 +29,7 @@
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  *
- *	$Id$
+ *	$Id: hash.c,v 1.1.1.1 1995/10/26 16:25:29 wpaul Exp $
  */
 
 #include <stdio.h>
@@ -38,6 +38,10 @@
 #include <sys/types.h>
 #include "hash.h"
 
+#ifndef lint
+static const char rcsid[] = "$Id$";
+#endif
+
 /*
  * This hash function is stolen directly from the
  * Berkeley DB package. It already exists inside libc, but
@@ -97,7 +101,7 @@ hash(keyarg, len)
 /*
  * Generate a hash value for a given key (character string).
  * We mask off all but the lower 8 bits since our table array
- * can only hole 256 elements.
+ * can only hold 256 elements.
  */
 u_int32_t hashkey(key)
 	char *key;
@@ -136,7 +140,7 @@ char *lookup(table, key)
  *
  * One way to deal with this is to malloc(2) a second table and start
  * doing indirection, but this is a pain in the butt and it's not worth
- * going to all that trouble for a dinky littke program like this. Instead,
+ * going to all that trouble for a dinky little program like this. Instead,
  * we turn each table entry into a linked list and simply link keys
  * with the same hash value together at the same index location within
  * the table.
@@ -162,7 +166,7 @@ void store (table, key, data)
 }
 
 /*
- * Store an group member entry and/or update its grouplist. This is
+ * Store a group member entry and/or update its grouplist. This is
  * a bit more complicated than the previous function since we have to
  * maintain not only the hash table of group members, each group member
  * structure also has a linked list of groups hung off it. If handed

libexec/revnetgroup/hash.h

@@ -29,7 +29,7 @@
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  *
- *	$Id$
+ *	$Id: hash.h,v 1.1.1.1 1995/10/26 16:25:29 wpaul Exp $
  */
 
 /* Groupname entry hung off a member_entry node. */
@@ -50,7 +50,6 @@ struct member_entry {
 struct group_entry {
 	char *key;
 	char *data;
-	struct grps *groups;
 	struct group_entry *next;
 };
 

libexec/revnetgroup/parse_netgroup.c

@@ -35,7 +35,7 @@
  */
 
 #if defined(LIBC_SCCS) && !defined(lint)
-static char sccsid[] = "$Id$";
+static char sccsid[] = "$Id: parse_netgroup.c,v 1.1.1.1 1995/10/26 16:25:29 wpaul Exp $";
 #endif /* LIBC_SCCS and not lint */
 
 /*
@@ -51,6 +51,10 @@ static char sccsid[] = "$Id$";
 #include <unistd.h>
 #include "hash.h"
 
+#ifndef lint
+static const char rcsid[] = "$Id$";
+#endif
+
 /*
  * Static Variables and functions used by setnetgrent(), getnetgrent() and
  * __endnetgrent().

libexec/revnetgroup/revnetgroup.c

@@ -35,7 +35,7 @@
  * Center for Telecommunications Research
  * Columbia University, New York City
  *
- *	$Id$
+ *	$Id: revnetgroup.c,v 1.1.1.1 1995/10/26 16:25:29 wpaul Exp $
  */
 
 #include <stdio.h>
@@ -43,6 +43,10 @@
 #include <string.h>
 #include "hash.h"
 
+#ifndef lint
+static const char rcsid[] = "$Id$";
+#endif
+
 #define LINSIZ 1024
 
 /* Default location of netgroup file. */
@@ -71,17 +75,17 @@ char *prog;
 extern char *optarg;
 
 main(argc, argv)
-int argc;
-char *argv[0];
+	int argc;
+	char *argv[];
 {
 	FILE *fp;
 	char readbuf[LINSIZ];
 	struct group_entry *gcur;
 	struct member_entry *mcur;
 	char *host, *user, *domain;
-	char ch;
+	int ch;
 	char *key = NULL, *data = NULL;
-	int hosts, i;
+	int hosts = -1, i;
 
 	if (argc < 2)
 		usage(argv[0]);
@@ -89,9 +93,17 @@ char *argv[0];
 	while ((ch = getopt(argc, argv, "uhf:")) != EOF) {
 		switch(ch) {
 		case 'u':
+			if (hosts != -1) {
+				warnx("please use only one of -u or -h");
+				usage(argv[0]);
+			}
 			hosts = 0;
 			break;
 		case 'h':
+			if (hosts != -1) {
+				warnx("please use only one of -u or -h");
+				usage(argv[0]);
+			}
 			hosts = 1;
 			break;
 		case 'f':
@@ -103,6 +115,9 @@ char *argv[0];
 		}
 	}
 
+	if (hosts == -1)
+		usage(argv[0]);
+
 	if (strcmp(netgroup, "-")) {
 		if ((fp = fopen(netgroup, "r")) == NULL) {
 			perror(netgroup);