Browse Source

nl80211: remove redundant check in nl80211_phy2ifname()

phyidx is already checked for <0 a few lines above.

Signed-off-by: Andre Heider <a.heider@gmail.com>
Andre Heider 1 year ago
parent
commit
acbf4fe47d
1 changed files with 13 additions and 16 deletions
  1. 13 16
      iwinfo_nl80211.c

+ 13 - 16
iwinfo_nl80211.c

@@ -826,31 +826,28 @@ static char * nl80211_phy2ifname(const char *ifname)
 
 	memset(nif, 0, sizeof(nif));
 
-	if (phyidx > -1)
+	if ((d = opendir("/sys/class/net")) != NULL)
 	{
-		if ((d = opendir("/sys/class/net")) != NULL)
+		while ((e = readdir(d)) != NULL)
 		{
-			while ((e = readdir(d)) != NULL)
+			snprintf(buffer, sizeof(buffer),
+			         "/sys/class/net/%s/phy80211/index", e->d_name);
+
+			if (nl80211_readint(buffer) == phyidx)
 			{
 				snprintf(buffer, sizeof(buffer),
-				         "/sys/class/net/%s/phy80211/index", e->d_name);
+				         "/sys/class/net/%s/ifindex", e->d_name);
 
-				if (nl80211_readint(buffer) == phyidx)
+				if ((cifidx = nl80211_readint(buffer)) >= 0 &&
+				    ((ifidx < 0) || (cifidx < ifidx)))
 				{
-					snprintf(buffer, sizeof(buffer),
-					         "/sys/class/net/%s/ifindex", e->d_name);
-
-					if ((cifidx = nl80211_readint(buffer)) >= 0 &&
-					    ((ifidx < 0) || (cifidx < ifidx)))
-					{
-						ifidx = cifidx;
-						strncpy(nif, e->d_name, sizeof(nif) - 1);
-					}
+					ifidx = cifidx;
+					strncpy(nif, e->d_name, sizeof(nif) - 1);
 				}
 			}
-
-			closedir(d);
 		}
+
+		closedir(d);
 	}
 
 	return nif[0] ? nif : NULL;