"Julian H. Stacey" <j...@berklix.com> wrote
  in <201310230254.r9n2sld1067...@fire.js.berklix.net>:

jh> Regresssion in 10.0-BETA1, mdconfig -l -v return code changed from 0
jh> to 255.  OK, its not specfied in manual, but has changed.
jh> A test script I was using caught it:
jh>     http://www.berklix.com/~jhs/bin/sh/mdconfig_tst
jh> Probably in next day or 2 I'll hack the C & send-pr.

 Does the attached patch fix it?

-- Hiroki
Index: sbin/mdconfig/mdconfig.c
===================================================================
--- sbin/mdconfig/mdconfig.c	(revision 256952)
+++ sbin/mdconfig/mdconfig.c	(working copy)
@@ -481,12 +481,18 @@
 		printf("\n");
 	/* XXX: Check if it's enough to clean everything. */
 	geom_stats_snapshot_free(sq);
-	if (((opt & OPT_UNIT) && (fflag == NULL) && ufound) ||
-	    ((opt & OPT_UNIT) == 0 && (fflag != NULL) && ffound) ||
-	    ((opt & OPT_UNIT) && (fflag != NULL) && ufound && ffound))
-		return (0);
-	else
-		return (-1);
+	if (opt & OPT_UNIT) {
+		if (((fflag == NULL) && ufound) ||
+		    ((fflag == NULL) && (units != NULL) && ufound) ||
+		    ((fflag != NULL) && ffound) ||
+		    ((fflag != NULL) && (units != NULL) && ufound && ffound))
+			return (0);
+	} else if (opt & OPT_LIST) {
+		if ((fflag == NULL) ||
+		    ((fflag != NULL) && ffound))
+			return (0);
+	}
+	return (-1);
 }

 /*

Attachment: pgpk_4KNOnPjY.pgp
Description: PGP signature

Reply via email to