From 909ebdde4ee2233d65de8fa01fde8e9a3bec12b7 Mon Sep 17 00:00:00 2001
From: Khem Raj <raj.khem@gmail.com>
Date: Tue, 27 Jun 2017 09:33:26 -0700
Subject: [PATCH 2/2] fix err() API to have format string

Fixes errors with hardening flags

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 host/tools/wmiconfig/wmiconfig.c | 148 +++++++++++++++++++--------------------
 1 file changed, 74 insertions(+), 74 deletions(-)

diff --git a/host/tools/wmiconfig/wmiconfig.c b/host/tools/wmiconfig/wmiconfig.c
index 21c9dcd..a6ec481 100644
--- a/wmiconfig.c
+++ b/wmiconfig.c
@@ -483,7 +483,7 @@ main (int argc, char **argv)
     strcpy(ifname, ethIf);
     s = socket(AF_INET, SOCK_DGRAM, 0);
     if (s < 0) {
-        err(1, "socket");
+        err(1, "%s", "socket");
     }
 
     while (1) {
@@ -1506,28 +1506,28 @@ main (int argc, char **argv)
         ifr.ifr_data = (void *)filterCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_SETBSSFILTER, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_POWER_MODE:
         ifr.ifr_data = (void *)pwrCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_SETPWR, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_PM_PARAMS:
         ifr.ifr_data = (void *)pmParamCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_SET_PMPARAMS, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_IBSS_PM_CAPS:
         ifr.ifr_data = (void *)adhocPmCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_SET_IBSS_PM_CAPS, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_ERROR_DETECTION:
@@ -1535,7 +1535,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_GET_HB_CHALLENGE_RESP:
@@ -1543,7 +1543,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
 #ifdef USER_KEYS
@@ -1554,7 +1554,7 @@ main (int argc, char **argv)
 
             if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
             {
-                err(1, ifr.ifr_name);
+                err(1, "%s", ifr.ifr_name);
             }
 
             break;
@@ -1575,7 +1575,7 @@ main (int argc, char **argv)
         ifr.ifr_data = (void *)sParamCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_SETSCAN, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_GET_VERSION:
@@ -1586,7 +1586,7 @@ main (int argc, char **argv)
         ifr.ifr_data = (void *)revinfo;
         if (ioctl(s, AR6000_IOCTL_WMI_GETREV, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         printf("Host Rev = 0x%x(%u.%u.%u.%u), Target Rev = 0x%x(%u.%u.%u.%u)\n",
               revinfo->host_ver,
@@ -1606,14 +1606,14 @@ main (int argc, char **argv)
         ifr.ifr_data = (void *)listenCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_SETLISTENINT, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_BMISS_TIME:
         ifr.ifr_data = (void *)bmissCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_SET_BMISS_TIME, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_RSSI_THRESHOLDS:
@@ -1621,14 +1621,14 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_SNR_THRESHOLDS:
         ifr.ifr_data = (void *)snrThresholdParam;
         if (ioctl(s, AR6000_IOCTL_WMI_SET_SNRTHRESHOLD, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_CLR_RSSISNR:
@@ -1636,7 +1636,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_LQ_THRESHOLDS:
@@ -1644,7 +1644,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_CHANNEL:
@@ -1693,18 +1693,18 @@ main (int argc, char **argv)
  
         if (ioctl(s, AR6000_IOCTL_WMI_SET_CHANNELPARAMS, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_SSID:
         if (index > MAX_PROBED_SSID_INDEX) {
             printf("num option for ssid command too large\n");
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
             break;
         }
         if (strlen((char *)ssid) > sizeof (ssidCmd->ssid)) {
             printf("ssid name too large\n");
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
             break;
         }
         ssidCmd->entryIndex = index;
@@ -1722,7 +1722,7 @@ main (int argc, char **argv)
         ifr.ifr_data = (void *)ssidCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_SET_PROBEDSSID, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_BADAP:
@@ -1735,7 +1735,7 @@ main (int argc, char **argv)
         ifr.ifr_data = (void *)badApCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_SET_BADAP, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_CREATE_QOS:
@@ -1794,7 +1794,7 @@ main (int argc, char **argv)
         ifr.ifr_data = (void *)crePStreamCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_CREATE_QOS, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_GET_TARGET_STATS:
@@ -1807,7 +1807,7 @@ main (int argc, char **argv)
         ifr.ifr_data = (void *)&tgtStatsCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_GET_TARGET_STATS, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         printTargetStats(&(tgtStatsCmd.targetStats));
         break;
@@ -1815,7 +1815,7 @@ main (int argc, char **argv)
         ifr.ifr_data = (void *)pBitMask;
         if (ioctl(s, AR6000_IOCTL_WMI_SET_ERROR_REPORT_BITMASK, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_DELETE_QOS:
@@ -1824,7 +1824,7 @@ main (int argc, char **argv)
         ifr.ifr_data = (void *)delPStreamCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_DELETE_QOS, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_GET_QOS_QUEUE:
@@ -1840,7 +1840,7 @@ main (int argc, char **argv)
         ifr.ifr_data = (void *)getQosQueueCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_GET_QOS_QUEUE, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
 
         printf("Active TSIDs \n");
@@ -1855,7 +1855,7 @@ main (int argc, char **argv)
         ifr.ifr_data = (void *)ieInfo;
         if (ioctl(s, AR6000_IOCTL_WMI_SET_ASSOC_INFO, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_AC_PARAMS:
@@ -1871,14 +1871,14 @@ main (int argc, char **argv)
         ifr.ifr_data = (void *)acParamsCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_SET_ACCESS_PARAMS, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_DISC_TIMEOUT:
         ifr.ifr_data = (void *)discCmd;
         if (ioctl(s, AR6000_IOCTL_WMI_SET_DISC_TIMEOUT, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
      case WMI_SET_ADHOC_BSSID:
@@ -1887,7 +1887,7 @@ main (int argc, char **argv)
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
             printf("fail to set adhoc bssid \n");
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_OPT_MODE:
@@ -1895,7 +1895,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_OPT_SEND_FRAME:
@@ -1903,7 +1903,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_BEACON_INT:
@@ -1911,7 +1911,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_VOICE_PKT_SIZE:
@@ -1919,7 +1919,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_MAX_SP:
@@ -1927,7 +1927,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_GET_ROAM_TBL:
@@ -1935,7 +1935,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_ROAM_CTRL:
@@ -1943,7 +1943,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_POWERSAVE_TIMERS:
@@ -1951,7 +1951,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_GET_POWER_MODE:
@@ -1959,7 +1959,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         printf("Power mode is %s\n",
                (getPowerMode->powerMode == MAX_PERF_POWER) ? "maxperf" : "rec");
@@ -1969,7 +1969,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_GET_ROAM_DATA:
@@ -1977,7 +1977,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_BT_STATUS:
@@ -1985,7 +1985,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_BT_PARAMS:
@@ -1993,7 +1993,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;   
     case WMI_SET_RETRYLIMITS:
@@ -2001,14 +2001,14 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_START_SCAN:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_STARTSCAN;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_FIX_RATES:
@@ -2031,14 +2031,14 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_GET_FIX_RATES:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_GETFIXRATES;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         } else {
             int i;
             printf("Fix rate set index:");
@@ -2057,7 +2057,7 @@ main (int argc, char **argv)
         index--;
         setAuthMode->mode = atoi(argv[index]);
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_REASSOC_MODE:
@@ -2067,42 +2067,42 @@ main (int argc, char **argv)
         index--;
         setReassocMode->mode = atoi(argv[index]);
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_LPREAMBLE:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_SET_LPREAMBLE;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_RTS:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_SET_RTS;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_WMM:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_SET_WMM;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_TXOP:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_SET_TXOP;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case DIAG_READ:
         ((int *)buf)[0] = AR6000_XIOCTL_DIAG_READ;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         printf("diagdata: 0x%x\n", *diagdata);
         break;
@@ -2110,7 +2110,7 @@ main (int argc, char **argv)
         ((int *)buf)[0] = AR6000_XIOCTL_DIAG_WRITE;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_GET_RD:
@@ -2118,7 +2118,7 @@ main (int argc, char **argv)
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0)
         {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         else
         {
@@ -2139,14 +2139,14 @@ main (int argc, char **argv)
         index--;
         setKeepAlive->keepaliveInterval = atoi(argv[index]);
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_GET_KEEPALIVE:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_GET_KEEPALIVE;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         printf("Keepalive interval is %d secs and AP is %s\n",
                getKeepAlive->keepaliveInterval, (getKeepAlive->configured ?
@@ -2156,63 +2156,63 @@ main (int argc, char **argv)
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_SET_APPIE;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_MGMT_FRM_RX_FILTER:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_SET_MGMT_FRM_RX_FILTER;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_DBGLOG_CFG_MODULE:
         ((int *)buf)[0] = AR6000_XIOCTL_DBGLOG_CFG_MODULE;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_DBGLOG_GET_DEBUG_LOGS:
         ((int *)buf)[0] = AR6000_XIOCTL_DBGLOG_GET_DEBUG_LOGS;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_HOST_SLEEP_MODE:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_SET_HOST_SLEEP_MODE;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_WOW_MODE:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_SET_WOW_MODE;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_ADD_WOW_PATTERN:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_ADD_WOW_PATTERN;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_DEL_WOW_PATTERN:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_DEL_WOW_PATTERN;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_GET_WOW_LIST:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_GET_WOW_LIST;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case DIAG_DUMP_CHIP_MEM:
@@ -2225,7 +2225,7 @@ main (int argc, char **argv)
                 ((int *)buf)[0] = AR6000_XIOCTL_DIAG_READ;
                 ifr.ifr_data = buf;
                 if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-                    err(1, ifr.ifr_name);
+                    err(1, "%s", ifr.ifr_name);
                 }
                 printf("0x%04x:0x%04x\n", *diagaddr, *diagdata);
             }
@@ -2237,21 +2237,21 @@ main (int argc, char **argv)
         index = optind - 1;
         *connectCtrlFlags = strtoul(argv[index], NULL, 0);
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case DUMP_HTC_CREDITS:        
         ((int *)buf)[0] = AR6000_XIOCTL_DUMP_HTC_CREDIT_STATE;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_AKMP_INFO:
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_SET_AKMP_PARAMS;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_PMKID_LIST:
@@ -2259,7 +2259,7 @@ main (int argc, char **argv)
             ((int *)buf)[0] = AR6000_XIOCTL_WMI_SET_PMKID_LIST;
             ifr.ifr_data = buf;
             if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-                err(1, ifr.ifr_name);
+                err(1, "%s", ifr.ifr_name);
             }
         } else {
             printf("No PMKIDs entered\n");
@@ -2269,7 +2269,7 @@ main (int argc, char **argv)
         ((int *)buf)[0] = AR6000_XIOCTL_WMI_GET_PMKID_LIST;
         ifr.ifr_data = buf;
         if (ioctl(s, AR6000_IOCTL_EXTENDED, &ifr) < 0) {
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     case WMI_SET_BSS_PMKID_INFO:
@@ -2277,7 +2277,7 @@ main (int argc, char **argv)
         iwr.u.data.length = sizeof(*pi_cmd);
         if (ioctl(s, IEEE80211_IOCTL_ADDPMKID, &iwr) < 0) {
             printf("ADDPMKID IOCTL Error\n");
-            err(1, ifr.ifr_name);
+            err(1, "%s", ifr.ifr_name);
         }
         break;
     default:
-- 
2.13.2