brcmfmac: check return for ARP ip setting iovar
The return value of iovar set function should be saved and checked. Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: Hante Meuleman <meuleman@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
This commit is contained in:
@@ -1039,16 +1039,16 @@ static int brcmf_inetaddr_changed(struct notifier_block *nb,
|
|||||||
return NOTIFY_OK;
|
return NOTIFY_OK;
|
||||||
}
|
}
|
||||||
for (i = 0; i < ARPOL_MAX_ENTRIES; i++) {
|
for (i = 0; i < ARPOL_MAX_ENTRIES; i++) {
|
||||||
if (addr_table[i] != 0) {
|
if (addr_table[i] == 0)
|
||||||
brcmf_fil_iovar_data_set(ifp,
|
continue;
|
||||||
"arp_hostip", &addr_table[i],
|
ret = brcmf_fil_iovar_data_set(ifp, "arp_hostip",
|
||||||
|
&addr_table[i],
|
||||||
sizeof(addr_table[i]));
|
sizeof(addr_table[i]));
|
||||||
if (ret)
|
if (ret)
|
||||||
brcmf_err("add arp ip err %d\n",
|
brcmf_err("add arp ip err %d\n",
|
||||||
ret);
|
ret);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
break;
|
break;
|
||||||
|
Reference in New Issue
Block a user