atactl: remove few printf("%s", NULL)

[Sebastien Marie]

Hi,

While looking at my worktree, I found some old changes I have locally
which could be commited.

Here one to remove few printf("%s", NULL) I saw long time ago in
atactl. I don't remember exactly when I saw them, which field trigger
syslog entry, and if I would still see them.

Comments or OK ?
-- 
Sebastien Marie



diff c9d76f17cafb34f4739ba697be46a54e300798de 
84d6df4e4168c371fb46e81b1c44c947a6b0b3f6
blob - 625e16f3087d2400445c1f69d41eb130703758b9
blob + b8858f311a522577502b0faf15a1c121e3525f8c
--- sbin/atactl/atactl.c
+++ sbin/atactl/atactl.c
@@ -74,7 +74,7 @@ __dead void usage(void);
 void ata_command(struct atareq *);
 void print_bitinfo(const char *, u_int, struct bitinfo *);
 int  strtoval(const char *, struct valinfo *);
-const char *valtostr(int, struct valinfo *);
+const char *valtostr(int, struct valinfo *, const char *);
 
 int    fd;                             /* file descriptor for device */
 
@@ -453,15 +453,15 @@ strtoval(const char *str, struct valinfo *vinfo)
 /*
  * valtostr():
  *    returns string associated with given value,
- *    if no string found NULL is returned.
+ *    if no string found def value is returned.
  */
 const char *
-valtostr(int val, struct valinfo *vinfo)
+valtostr(int val, struct valinfo *vinfo, const char *def)
 {
        for (; vinfo->string != NULL; vinfo++)
                if (val == vinfo->value)
                        return (vinfo->string);
-       return (NULL);
+       return (def);
 }
 
 /*
@@ -1338,14 +1338,14 @@ device_smart_read(int argc, char *argv[])
 
        printf("Off-line data collection:\n");
        printf("    status: %s\n",
-           valtostr(data.offstat & 0x7f, smart_offstat));
+           valtostr(data.offstat & 0x7f, smart_offstat, "?"));
        printf("    activity completion time: %d seconds\n",
            letoh16(data.time));
        printf("    capabilities:\n");
        print_bitinfo("\t%s\n", data.offcap, smart_offcap);
        printf("Self-test execution:\n");
        printf("    status: %s\n", valtostr(SMART_SELFSTAT_STAT(data.selfstat),
-           smart_selfstat));
+           smart_selfstat, "?"));
        if (SMART_SELFSTAT_STAT(data.selfstat) == SMART_SELFSTAT_PROGRESS)
                printf("remains %d%% of total time\n",
                    SMART_SELFSTAT_PCNT(data.selfstat));
@@ -1511,7 +1511,7 @@ device_smart_readlog(int argc, char *argv[])
                        printf("    status: %s\n",
                            valtostr(SMART_SELFSTAT_STAT(
                            data->desc[i].selfstat),
-                           smart_selfstat));
+                           smart_selfstat, "?"));
                        printf("    timestamp: %d\n",
                            MAKEWORD(data->desc[i].time1,
                                     data->desc[i].time2));
@@ -1551,7 +1551,7 @@ smart_print_errdata(struct smart_log_errdata *data)
        printf("    LBA High register: 0x%x\n", data->err.reg_lbahi);
        printf("    device register: 0x%x\n", data->err.reg_dev);
        printf("    status register: 0x%x\n", data->err.reg_stat);
-       printf("    state: %s\n", valtostr(data->err.state, smart_logstat));
+       printf("    state: %s\n", valtostr(data->err.state, smart_logstat, 
"?"));
        printf("    timestamp: %d\n", MAKEWORD(data->err.time1,
                                               data->err.time2));
        printf("    history:\n");
@@ -1643,9 +1643,8 @@ device_attr(int argc, char *argv[])
        printf("ID\tAttribute name\t\t\tThreshold\tValue\tRaw\n");
        for (i = 0; i < 30; i++) {
                if (thr[i].id != 0 && thr[i].id == attr[i].id) {
-                       attr_name = valtostr(thr[i].id, ibm_attr_names);
-                       if (attr_name == NULL)
-                               attr_name = "Unknown";
+                       attr_name = valtostr(thr[i].id, ibm_attr_names,
+                           "Unknown");
 
                        for (k = 0; k < 6; k++) {
                                u_int8_t b;