12533 cfgadm_plugins: NULL pointer errors

Reviewed by: Gary Mills <gary_mills@fastmail.fm>
Approved by: Dan McDonald <danmcd@joyent.com>

2 files changed, 40 insertions, 41 deletions

diff --git a/usr/src/lib/cfgadm_plugins/ac/common/mema.c b/usr/src/lib/cfgadm_plugins/ac/common/mema.c
index 1b08d13f92..3e97bc622e 100644
--- a/usr/src/lib/cfgadm_plugins/ac/common/mema.c
+++ b/usr/src/lib/cfgadm_plugins/ac/common/mema.c
@@ -24,8 +24,6 @@
  * Use is subject to license terms.
  */
 
-#pragma ident	"%Z%%M%	%I%	%E% SMI"
-
 #include <stddef.h>
 #include <stdlib.h>
 #include <unistd.h>
@@ -465,7 +463,7 @@ ap_stat(
 
 	if ((bank = ap_bk_idx(bank_spec)) == -1) {
 		__fmt_errstring(errstring, strlen(bank_spec),
-			dgettext(TEXT_DOMAIN, ap_invalid), bank_spec);
+		    dgettext(TEXT_DOMAIN, ap_invalid), bank_spec);
 		return (CFGA_ERROR);
 	}
 
@@ -831,7 +829,7 @@ mtest_run(
 	if (ret == -1) {
 		if (ret_errno == ENOTSUP) {
 			mema_err(&cmd, ret_errno, errstring,
-				CMD_MEM_TEST_START);
+			    CMD_MEM_TEST_START);
 			return (CFGA_OPNOTSUPP);
 		}
 		if (ret_errno == EBUSY && test_start.tester_pid > 0) {
@@ -1131,14 +1129,14 @@ cfga_test(
 				hlp = subopt_help_str(opts);
 				if (hlp != NULL) {
 					__fmt_errstring(errstring,
-					strlen(value) + strlen(hlp),
-					dgettext(TEXT_DOMAIN, unk_test),
-					value, hlp);
+					    strlen(value) + strlen(hlp),
+					    dgettext(TEXT_DOMAIN, unk_test),
+					    value, hlp);
 					free((void *)hlp);
 				} else {
 					__fmt_errstring(errstring, 20,
-					dgettext(TEXT_DOMAIN, calloc_fail),
-						strlen(options) + 1, 1);
+					    dgettext(TEXT_DOMAIN, calloc_fail),
+					    strlen(options) + 1, 1);
 				}
 				/* Free after printing value. */
 				free((void *)free_cp);
@@ -1161,18 +1159,18 @@ cfga_test(
 
 				if (max_errors != -1 && subopt == maxerr_idx) {
 					__fmt_errstring(errstring,
-					strlen(opts[subopt]),
-					dgettext(TEXT_DOMAIN, dup_num),
-					opts[subopt]);
+					    strlen(opts[subopt]),
+					    dgettext(TEXT_DOMAIN, dup_num),
+					    opts[subopt]);
 					free((void *)free_cp);
 					return (CFGA_ERROR);
 				}
 
 				if (value == NULL) {
 					__fmt_errstring(errstring,
-					0,
-					dgettext(TEXT_DOMAIN, no_num),
-					"");
+					    0,
+					    dgettext(TEXT_DOMAIN, no_num),
+					    "");
 					free((void *)free_cp);
 					return (CFGA_ERROR);
 				}
@@ -1181,9 +1179,9 @@ cfga_test(
 				if ((ret_p == value) || (*ret_p != '\0') ||
 				    (max_errors < 0)) {
 					__fmt_errstring(errstring,
-					strlen(value),
-					dgettext(TEXT_DOMAIN, no_num),
-					value);
+					    strlen(value),
+					    dgettext(TEXT_DOMAIN, no_num),
+					    value);
 					free((void *)free_cp);
 					return (CFGA_ERROR);
 				}
@@ -1300,7 +1298,7 @@ info_set(ac_stat_t *asp, mema_bank_t *bkp, cfga_info_t info)
 	char *end;
 
 	end = &info[sizeof (cfga_info_t)];
-	*info = NULL;
+	*info = '\0';
 
 	board = bkp->board;
 
@@ -1370,8 +1368,8 @@ mema_cvt(ac_stat_t *ac, mema_bank_t *bkp, cfga_stat_data_t *cs)
 	cs->ap_busy = (cfga_busy_t)ac->busy;
 	cs->ap_status_time = ac->status_time;
 	info_set(ac, bkp, cs->ap_info);
-	cs->ap_log_id[0] = NULL;
-	cs->ap_phys_id[0] = NULL;
+	cs->ap_log_id[0] = '\0';
+	cs->ap_phys_id[0] = '\0';
 }
 
 /*ARGSUSED*/
diff --git a/usr/src/lib/cfgadm_plugins/sysctrl/common/cfga.c b/usr/src/lib/cfgadm_plugins/sysctrl/common/cfga.c
index 1dca716564..81245635d2 100644
--- a/usr/src/lib/cfgadm_plugins/sysctrl/common/cfga.c
+++ b/usr/src/lib/cfgadm_plugins/sysctrl/common/cfga.c
@@ -24,8 +24,6 @@
  * Use is subject to license terms.
  */
 
-#pragma ident	"%Z%%M%	%I%	%E% SMI"
-
 #include <stddef.h>
 #include <locale.h>
 #include <ctype.h>
@@ -611,7 +609,10 @@ sim_open(char *a, int b, int c)
 }
 
 static int
-sim_close(int a) { return (0); }
+sim_close(int a)
+{
+	return (0);
+}
 
 static int
 sim_ioctl(int fd, int cmd, void *a)
@@ -750,7 +751,7 @@ dlist_find(int board, char **dlist, int *disabled)
 
 static int
 dlist_update(int board, int disable, char *dlist, struct cfga_msg *msgp,
-	int verbose)
+    int verbose)
 {
 	int i, j, n;
 	int err;
@@ -941,7 +942,7 @@ cfga_change_state(
 			cfga_err(NULL, errstring, ERR_TRANS, 0);
 		else if (disabled && !(force || (options && !disable)))
 			cfga_err(NULL, errstring, CMD_CONNECT,
-				ERR_DISABLED, DIAG_FORCE, 0);
+			    ERR_DISABLED, DIAG_FORCE, 0);
 		else if (!(*confp->confirm)(confp->appdata_ptr,
 		    cfga_str(ASK_CONNECT))) {
 			(void) close(fd);
@@ -949,7 +950,7 @@ cfga_change_state(
 		} else if (ioctl(fd, SYSC_CFGA_CMD_CONNECT, sc) == -1)
 			cfga_err(sc, errstring, CMD_CONNECT, 0);
 		else if (options && (opterr = dlist_update(idx, disable,
-			dlist, msgp, verbose))) {
+		    dlist, msgp, verbose))) {
 			err = disable ? OPT_DISABLE : OPT_ENABLE;
 			cfga_err(NULL, errstring, err, opterr, 0);
 		} else
@@ -982,7 +983,7 @@ cfga_change_state(
 		if (rs == SYSC_CFGA_RSTATE_DISCONNECTED)
 			if (disabled && !(force || (options && !disable))) {
 				cfga_err(NULL, errstring, CMD_CONFIGURE,
-					ERR_DISABLED, DIAG_FORCE, 0);
+				    ERR_DISABLED, DIAG_FORCE, 0);
 				(void) close(fd);
 				return (CFGA_ERROR);
 			} else if (!(*confp->confirm)(confp->appdata_ptr,
@@ -1000,7 +1001,7 @@ cfga_change_state(
 			if (ioctl(fd, SYSC_CFGA_CMD_CONFIGURE, sc) == -1)
 				cfga_err(sc, errstring, CMD_CONFIGURE, 0);
 			else if (options && (opterr = dlist_update(idx,
-				disable, dlist, msgp, verbose))) {
+			    disable, dlist, msgp, verbose))) {
 				err = disable ? OPT_DISABLE : OPT_ENABLE;
 				cfga_err(NULL, errstring, err, opterr, 0);
 			} else
@@ -1015,7 +1016,7 @@ cfga_change_state(
 		else if (ioctl(fd, SYSC_CFGA_CMD_UNCONFIGURE, sc) == -1)
 			cfga_err(sc, errstring, CMD_UNCONFIGURE, 0);
 		else if (options && (opterr = dlist_update(idx, disable,
-			dlist, msgp, verbose))) {
+		    dlist, msgp, verbose))) {
 			err = disable ? OPT_DISABLE : OPT_ENABLE;
 			cfga_err(NULL, errstring, err, opterr, 0);
 		} else
@@ -1124,14 +1125,14 @@ cfga_private_func(
 	if ((idx = ap_idx(ap_id)) == -1)
 		cfga_err(NULL, errstring, ERR_AP_INVAL, ap_id, 0);
 	else if (((fd = open(ap_id, O_RDWR, 0)) == -1) ||
-		(ioctl(fd, cmd, sc) == -1))
+	    (ioctl(fd, cmd, sc) == -1))
 		cfga_err(NULL, errstring, err, 0);
 	else
 		rc = CFGA_OK;
 
 	if (options) {
 		opterr = (dlist_find(idx, &dlist, &disabled) ||
-			dlist_update(idx, disable, dlist, msgp, verbose));
+		    dlist_update(idx, disable, dlist, msgp, verbose));
 		if (opterr) {
 			err = disable ? OPT_DISABLE : OPT_ENABLE;
 			if (verbose)
@@ -1274,7 +1275,7 @@ info_set(sysc_cfga_stat_t *sc, cfga_info_t info, int disabled)
 	struct cpu_info *cpu;
 	union bd_un *bd = &sc->bd;
 
-	*info = NULL;
+	*info = '\0';
 
 	switch (sc->type) {
 	case CPU_BOARD:
@@ -1282,11 +1283,11 @@ info_set(sysc_cfga_stat_t *sc, cfga_info_t info, int disabled)
 			if (cpu->cpu_speed > 1) {
 				info += sprintf(info, "cpu %d: ", i);
 				info += sprintf(info, "%3d MHz ",
-						cpu->cpu_speed);
+				    cpu->cpu_speed);
 				if (cpu->cache_size)
 					info += sprintf(info, "%0.1fM ",
-						(float)cpu->cache_size /
-						(float)(1024 * 1024));
+					    (float)cpu->cache_size /
+					    (float)(1024 * 1024));
 			}
 		}
 		break;
@@ -1312,7 +1313,7 @@ info_set(sysc_cfga_stat_t *sc, cfga_info_t info, int disabled)
 		for (i = 0; i < 2; i++)
 			if (bd->dsk.disk_pres[i])
 				info += sprintf(info, "target: %2d ",
-						bd->dsk.disk_id[i]);
+				    bd->dsk.disk_id[i]);
 			else
 				info += sprintf(info, "no disk   ");
 		break;
@@ -1338,8 +1339,8 @@ sysc_cvt(sysc_cfga_stat_t *sc, cfga_stat_data_t *cs, int disabled)
 	cs->ap_busy = (cfga_busy_t)sc->in_transition;
 	cs->ap_status_time = sc->last_change;
 	info_set(sc, cs->ap_info, disabled);
-	cs->ap_log_id[0] = NULL;
-	cs->ap_phys_id[0] = NULL;
+	cs->ap_log_id[0] = '\0';
+	cs->ap_phys_id[0] = '\0';
 }
 
 /*ARGSUSED*/
@@ -1426,8 +1427,8 @@ cfga_stat(
 		rc = CFGA_OK;
 
 		if (options && options[0] && ((opterr != 0) ||
-			((opterr = dlist_update(idx, disable, dlist, NULL, 0))
-			!= 0))) {
+		    ((opterr = dlist_update(idx, disable, dlist, NULL, 0))
+		    != 0))) {
 				err = disable ? OPT_DISABLE : OPT_ENABLE;
 				cfga_err(NULL, errstring, err, opterr, 0);
 		}