mac80211_hwsim: fix check for custom world regdom array size
David Binderman reports that the conditions in the first loop are the wrong way around - checking the array contents before the size. Instead of leaving the empty loop there and reordering the two checks unify it into a single loop that skips over non-matches and exits after the first match. Reported-by: David Binderman <dcb314@hotmail.com> Signed-off-by: Johannes Berg <johannes.berg@intel.com>
This commit is contained in:
parent
8d819a92cc
commit
5875755c57
|
@ -2150,14 +2150,14 @@ static int append_radio_msg(struct sk_buff *skb, int id,
|
|||
if (param->regd) {
|
||||
int i;
|
||||
|
||||
for (i = 0; hwsim_world_regdom_custom[i] != param->regd &&
|
||||
i < ARRAY_SIZE(hwsim_world_regdom_custom); i++)
|
||||
;
|
||||
for (i = 0; i < ARRAY_SIZE(hwsim_world_regdom_custom); i++) {
|
||||
if (hwsim_world_regdom_custom[i] != param->regd)
|
||||
continue;
|
||||
|
||||
if (i < ARRAY_SIZE(hwsim_world_regdom_custom)) {
|
||||
ret = nla_put_u32(skb, HWSIM_ATTR_REG_CUSTOM_REG, i);
|
||||
if (ret < 0)
|
||||
return ret;
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue