iwinfo: Fix buffer size

[oweals/iwinfo.git] / iwinfo_nl80211.c

diff --git a/iwinfo_nl80211.c b/iwinfo_nl80211.c
index 13c69e3112292db0a513590ca1427904e6bca81c..690f8b2056a19fdc1107e20fccf49d0548e1fd79 100644 (file)
--- a/iwinfo_nl80211.c
+++ b/iwinfo_nl80211.c
@@ -1207,7 +1207,7 @@ static int nl80211_get_frequency_info_cb(struct nl_msg *msg, void *arg)
 
 static int nl80211_get_frequency(const char *ifname, int *buf)
 {
-       char *res, channel[4], hwmode[2];
+       char *res, channel[4], hwmode[3];
 
        /* try to find frequency from interface info */
        res = nl80211_phy2ifname(ifname);
@@ -1494,7 +1494,7 @@ static int nl80211_check_wepkey(const char *key)
 
 static int nl80211_get_encryption(const char *ifname, char *buf)
 {
-       char wpa[2], wpa_key_mgmt[16], wpa_pairwise[16], wpa_groupwise[16];
+       char wpa[2], wpa_key_mgmt[64], wpa_pairwise[16], wpa_groupwise[16];
        char auth_algs[2], wep_key0[27], wep_key1[27], wep_key2[27], wep_key3[27];
 
        struct iwinfo_crypto_entry *c = (struct iwinfo_crypto_entry *)buf;
@@ -1588,6 +1588,16 @@ static int nl80211_get_encryption(const char *ifname, char *buf)
                        if (strstr(wpa_key_mgmt, "EAP"))
                                c->auth_suites |= IWINFO_KMGMT_8021x;
 
+                       if (strstr(wpa_key_mgmt, "SAE")){
+                               c->auth_suites |= IWINFO_KMGMT_SAE;
+                               c->wpa_version = 4;
+                       }
+
+                       if (strstr(wpa_key_mgmt, "OWE")){
+                               c->auth_suites |= IWINFO_KMGMT_OWE;
+                               c->wpa_version = 4;
+                       }
+
                        if (strstr(wpa_key_mgmt, "NONE"))
                                c->auth_suites |= IWINFO_KMGMT_NONE;
                }
@@ -2926,37 +2936,37 @@ static int nl80211_get_mbssid_support(const char *ifname, int *buf)
 
 static int nl80211_get_hardware_id(const char *ifname, char *buf)
 {
-       int rv = -1;
-       char *res;
+       struct iwinfo_hardware_id *id = (struct iwinfo_hardware_id *)buf;
+       char *phy, num[8], path[PATH_MAX];
+       int i;
 
-       /* Got a radioX pseudo interface, find some interface on it or create one */
-       if (!strncmp(ifname, "radio", 5))
-       {
-               /* Reuse existing interface */
-               if ((res = nl80211_phy2ifname(ifname)) != NULL)
-               {
-                       rv = wext_ops.hardware_id(res, buf);
-               }
+       struct { const char *path; uint16_t *dest; } lookup[] = {
+               { "vendor", &id->vendor_id },
+               { "device", &id->device_id },
+               { "subsystem_vendor", &id->subsystem_vendor_id },
+               { "subsystem_device", &id->subsystem_device_id }
+       };
 
-               /* Need to spawn a temporary iface for finding IDs */
-               else if ((res = nl80211_ifadd(ifname)) != NULL)
-               {
-                       rv = wext_ops.hardware_id(res, buf);
-                       nl80211_ifdel(res);
-               }
-       }
-       else
+       memset(id, 0, sizeof(*id));
+
+       /* Try to determine the phy name from the given interface */
+       phy = nl80211_ifname2phy(ifname);
+
+       for (i = 0; i < ARRAY_SIZE(lookup); i++)
        {
-               rv = wext_ops.hardware_id(ifname, buf);
+               snprintf(path, sizeof(path), "/sys/class/%s/%s/device/%s",
+                        phy ? "ieee80211" : "net",
+                        phy ? phy : ifname, lookup[i].path);
+
+               if (nl80211_readstr(path, num, sizeof(num)) > 0)
+                       *lookup[i].dest = strtoul(num, NULL, 16);
        }
 
        /* Failed to obtain hardware IDs, search board config */
-       if (rv)
-       {
-               rv = iwinfo_hardware_id_from_mtd((struct iwinfo_hardware_id *)buf);
-       }
+       if (id->vendor_id == 0 || id->device_id == 0)
+               return iwinfo_hardware_id_from_mtd(id);
 
-       return rv;
+       return 0;
 }
 
 static const struct iwinfo_hardware_entry *