Commit fed48250 authored by Andrew Zaborowski's avatar Andrew Zaborowski Committed by Johannes Berg

mac80211_hwsim: Fix radio dump for radio idx 0

Since 6335698e the radio with idx of 0
will not get dumped in HWSIM_CMD_GET_RADIO because of the last_idx
checks.  Offset cb->args[0] by 1 similarly to what is done in nl80211.c.

Fixes: 6335698e ("mac80211_hwsim: add generation count for netlink dump operation")
Signed-off-by: default avatarAndrew Zaborowski <andrew.zaborowski@intel.com>
Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent 5247a77c
...@@ -3340,7 +3340,7 @@ static int hwsim_get_radio_nl(struct sk_buff *msg, struct genl_info *info) ...@@ -3340,7 +3340,7 @@ static int hwsim_get_radio_nl(struct sk_buff *msg, struct genl_info *info)
static int hwsim_dump_radio_nl(struct sk_buff *skb, static int hwsim_dump_radio_nl(struct sk_buff *skb,
struct netlink_callback *cb) struct netlink_callback *cb)
{ {
int last_idx = cb->args[0]; int last_idx = cb->args[0] - 1;
struct mac80211_hwsim_data *data = NULL; struct mac80211_hwsim_data *data = NULL;
int res = 0; int res = 0;
void *hdr; void *hdr;
...@@ -3368,7 +3368,7 @@ static int hwsim_dump_radio_nl(struct sk_buff *skb, ...@@ -3368,7 +3368,7 @@ static int hwsim_dump_radio_nl(struct sk_buff *skb,
last_idx = data->idx; last_idx = data->idx;
} }
cb->args[0] = last_idx; cb->args[0] = last_idx + 1;
/* list changed, but no new element sent, set interrupted flag */ /* list changed, but no new element sent, set interrupted flag */
if (skb->len == 0 && cb->prev_seq && cb->seq != cb->prev_seq) { if (skb->len == 0 && cb->prev_seq && cb->seq != cb->prev_seq) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment