quot: Use getopt(3) and show usage() if no arguments

Also update the man page and usage to be a little more accurate with the
-a flag.

Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/983

usr.sbin/quot/quot.8

@@ -35,8 +35,8 @@
 .Nd display disk space occupied by each user
 .Sh SYNOPSIS
 .Nm
-.Op Fl acfhknv
+.Op Fl cfhknv
-.Op Ar filesystem ...
+.Op Fl a | Ar filesystem ...
 .Sh DESCRIPTION
 The
 .Nm

usr.sbin/quot/quot.c

@@ -526,9 +526,9 @@ static void
 usage(void)
 {
 #ifdef	COMPAT
-	fprintf(stderr,"usage: quot [-nfcvha] [filesystem ...]\n");
+	fprintf(stderr, "usage: quot [-cfhnv] [-a | filesystem ...]\n");
 #else	/* COMPAT */
-	fprintf(stderr,"usage: quot [-acfhknv] [filesystem ...]\n");
+	fprintf(stderr, "usage: quot [-cfhknv] [-a | filesystem ...]\n");
 #endif	/* COMPAT */
 	exit(1);
 }
@@ -575,23 +575,20 @@ main(int argc, char *argv[])
 	struct statfs *mp;
 	struct fstab *fs;
 	int cnt;
+	int ch;
 
 	func = douser;
 #ifndef	COMPAT
 	header = getbsize(&headerlen,&blocksize);
 #endif
-	while (--argc > 0 && **++argv == '-') {
+	while ((ch = getopt(argc, argv, "acfhknv")) != -1) {
-		while (*++*argv) {
+		switch (ch) {
-			switch (**argv) {
+		case 'a':
-			case 'n':
+			all = 1;
-				func = donames;
 			break;
 		case 'c':
 			func = dofsizes;
 			break;
-			case 'a':
-				all = 1;
-				break;
 		case 'f':
 			count = 1;
 			break;
@@ -603,6 +600,9 @@ main(int argc, char *argv[])
 			blocksize = 1024;
 			break;
 #endif	/* COMPAT */
+		case 'n':
+			func = donames;
+			break;
 		case 'v':
 			unused = 1;
 			break;
@@ -610,7 +610,12 @@ main(int argc, char *argv[])
 			usage();
 		}
 	}
-	}
+	argc -= optind;
+	argv += optind;
+
+	if ((argc == 0 && !all) || (all && argc))
+		usage();
+
 	if (all) {
 		cnt = getmntinfo(&mp,MNT_NOWAIT);
 		for (; --cnt >= 0; mp++) {