Mercurial > illumos > illumos-gate

changeset 13467:7ad3a0a0b2cb

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
1551 svccfg should not treat NULL as an integer Reviewed by: Joshua M. Clulow <josh@sysmgr.org> Reviewed by: Alexander Eremin <alexander.eremin@nexenta.com> Reviewed by: Igor Kozhukhov <ikozhukhov@gmail.com> Approved by: Garrett D'Amore <garrett@nexenta.com>
author Richard Lowe <richlowe@richlowe.net>
date Mon, 16 May 2011 07:03:03 +0100
parents e23b8d796527
children bf3e5d60b8e6
files usr/src/cmd/svc/svccfg/svccfg_libscf.c
diffstat 1 files changed, 7 insertions(+), 7 deletions(-) [+]
line wrap: on
line diff
--- a/usr/src/cmd/svc/svccfg/svccfg_libscf.c	Tue Jun 21 04:52:34 2011 -0700
+++ b/usr/src/cmd/svc/svccfg/svccfg_libscf.c	Mon May 16 07:03:03 2011 +0100
@@ -13070,7 +13070,7 @@
 		abort();
 
 	/* If pattern is specified, we want to list only those items. */
-	while (scf_tmpl_iter_pgs(pgt, fmribuf, snapbuf, NULL, NULL) == 1) {
+	while (scf_tmpl_iter_pgs(pgt, fmribuf, snapbuf, NULL, 0) == 1) {
 		listed = 0;
 		if (pattern == NULL || (scf_tmpl_pg_name(pgt, &pg_name) > 0 &&
 		    fnmatch(pattern, pg_name, 0) == 0)) {
@@ -13080,7 +13080,7 @@
 
 		scf_tmpl_prop_reset(prt);
 
-		while (scf_tmpl_iter_props(pgt, prt, NULL) == 0) {
+		while (scf_tmpl_iter_props(pgt, prt, 0) == 0) {
 			search_name = NULL;
 			prop_name_size = scf_tmpl_prop_name(prt, &prop_name);
 			if ((prop_name_size > 0) && (pg_name != NULL)) {
@@ -13192,7 +13192,7 @@
 			scfdie();
 		assert(szret <= pgnlen);
 
-		if (scf_tmpl_get_by_pg(pg, pgt, NULL) == -1) {
+		if (scf_tmpl_get_by_pg(pg, pgt, 0) == -1) {
 			if (scf_error() != SCF_ERROR_NOT_FOUND)
 				scfdie();
 			pgtp = NULL;
@@ -13284,7 +13284,7 @@
 
 				if (pgtp != NULL) {
 					if (scf_tmpl_get_by_prop(pgt, prnbuf,
-					    prt, NULL) < 0) {
+					    prt, 0) < 0) {
 						if (scf_error() !=
 						    SCF_ERROR_NOT_FOUND)
 							scfdie();
@@ -13633,9 +13633,9 @@
 
 				/* Then check for a type set in a template. */
 				} else if (scf_tmpl_get_by_pg(pg, pgt,
-				    NULL) == 0 &&
+				    0) == 0 &&
 				    scf_tmpl_get_by_prop(pgt, propname, prt,
-				    NULL) == 0 &&
+				    0) == 0 &&
 				    scf_tmpl_prop_type(prt, &current_ty) == 0) {
 					ty = current_ty;
 
@@ -15522,7 +15522,7 @@
 		}
 
 		for (p = pgs; *p; ++p) {
-			if (smf_notify_del_params(de_tag(*p), NULL, NULL) !=
+			if (smf_notify_del_params(de_tag(*p), NULL, 0) !=
 			    SCF_SUCCESS) {
 				uu_warn(gettext("Failed for \"%s\": %s\n"), *p,
 				    scf_strerror(scf_error()));