X-Git-Url: http://git.archive.openwrt.org/?p=project%2Fubox.git;a=blobdiff_plain;f=validate%2Fcli.c;h=f03c8a16f38dfb276e1633b5d1a21da876ebad59;hp=935b5f724881e00bfb018276854de78832d4eb4d;hb=8488bb5bfdaf6ab4771622c57a62aad5fd491f06;hpb=89fa6427a27091dd501b66b9bcf009254abd7d80

diff --git a/validate/cli.c b/validate/cli.c
index 935b5f7..f03c8a1 100644
--- a/validate/cli.c
+++ b/validate/cli.c
@@ -105,43 +105,32 @@ export_value(enum dt_type type, const char *name, const char *val)
 	printf("; ");
 }
 
-static void
+static int
 validate_value(struct uci_ptr *ptr, const char *expr, const char *def)
 {
 	int i = 0;
-	bool empty = true, first = true;
-	enum dt_type type;
+	bool empty = true;
+	enum dt_type type = DT_INVALID;
 	struct uci_element *e;
-	struct uci_option *opt = ptr->o;
-
-	if (opt->type == UCI_TYPE_LIST)
-	{
-		uci_foreach_element(&opt->v.list, e)
-		{
-			if (!e->name || !*e->name)
-				continue;
-
-			empty = false;
-			break;
-		}
+	struct uci_option *opt = NULL;
 
-		if (empty)
-		{
-			export_value(DT_STRING, ptr->option, def);
-			return;
-		}
+	if ((ptr->flags & UCI_LOOKUP_COMPLETE) &&
+	    (ptr->last->type == UCI_TYPE_OPTION))
+		opt = ptr->o;
 
+	if (opt && opt->type == UCI_TYPE_LIST)
+	{
 		uci_foreach_element(&opt->v.list, e)
 		{
 			if (!e->name || !*e->name)
 				continue;
 
-			if (first)
+			if (empty)
 				printf("%s=", ptr->option);
 			else
 				printf("\\ ");
 
-			first = false;
+			empty = false;
 			type = dt_parse(expr, e->name);
 
 			if (type != DT_INVALID)
@@ -152,16 +141,12 @@ validate_value(struct uci_ptr *ptr, const char *expr, const char *def)
 			        expr, type ? "true" : "false");
 		}
 
-		printf("; ");
+		if (!empty)
+			printf("; ");
 	}
-	else
+	else if (opt && opt->v.string && *opt->v.string)
 	{
-		if (!opt->v.string || !*opt->v.string)
-		{
-			export_value(DT_STRING, ptr->option, def);
-			return;
-		}
-
+		empty = false;
 		type = dt_parse(expr, opt->v.string);
 		export_value(type, ptr->option, opt->v.string);
 
@@ -169,9 +154,24 @@ validate_value(struct uci_ptr *ptr, const char *expr, const char *def)
 				ptr->package, ptr->section, ptr->option, opt->v.string,
 		        expr, type ? "true" : "false");
 	}
+
+	if (empty)
+	{
+		type = dt_parse(expr, def);
+
+		if (type == DT_INVALID)
+			type = DT_STRING;
+
+		export_value(type, ptr->option, def);
+
+		fprintf(stderr, "%s.%s.%s is unset and defaults to %s %s\n",
+				ptr->package, ptr->section, ptr->option, expr, def);
+	}
+
+	return type ? 0 : -1;
 }
 
-static void
+static int
 validate_option(struct uci_context *ctx, char *package, char *section, char *option)
 {
 	char *opt, *expr, *def;
@@ -180,22 +180,16 @@ validate_option(struct uci_context *ctx, char *package, char *section, char *opt
 	if (!parse_tuple(option, &opt, &expr, &def))
 	{
 		fprintf(stderr, "%s is not a valid option\n", option);
-		return;
+		return -1;
 	}
 
 	ptr.package = package;
 	ptr.section = section;
 	ptr.option = opt;
 
-	if (uci_lookup_ptr(ctx, &ptr, NULL, false) ||
-	    !(ptr.flags & UCI_LOOKUP_COMPLETE) ||
-	    (ptr.last->type != UCI_TYPE_OPTION))
-	{
-		export_value(DT_STRING, opt, def);
-		return;
-	}
+	uci_lookup_ptr(ctx, &ptr, NULL, false);
 
-	validate_value(&ptr, expr, def);
+	return validate_value(&ptr, expr, def);
 }
 
 int
@@ -206,7 +200,7 @@ main(int argc, char **argv)
 	char *opt, *expr, *def;
 	int len = argc - 4;
 	enum dt_type rv;
-	int i;
+	int i, rc;
 
 	if (argc == 3) {
 		rv = dt_parse(argv[1], argv[2]);
@@ -243,8 +237,12 @@ main(int argc, char **argv)
 	if (uci_load(ctx, argv[1], &package))
 		return -1;
 
-	for (i = 0; i < len; i++)
-		validate_option(ctx, argv[1], argv[3], argv[4 + i]);
+	rc = 0;
+	for (i = 0; i < len; i++) {
+		if (validate_option(ctx, argv[1], argv[3], argv[4 + i])) {
+			rc = -1;
+		}
+	}
 
-	return 0;
+	return rc;
 }