jeltz/hostap
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

FST: Remove unnecessary fst_attach() already attached check

Browse source 
Now that both hostapd and wpa_supplicant already enforce no duplicate
fst_attach() calls, there is no need for this check within fst_attach().

Signed-off-by: Jouni Malinen <j@w1.fi>
This commit is contained in:
Jouni Malinen 2015-07-26 12:10:14 +03:00
parent 5994658309
commit 55cd405a44
1 changed files with 0 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
src/fst/fst.c
View file

@ -34,19 +34,6 @@ static void fst_ctrl_iface_notify_peer_state_change(struct fst_iface *iface,
} }
static struct fst_group * fst_find_group_by_iface(const char *ifname)
{
struct fst_group *g;
foreach_fst_group(g) {
if (fst_group_get_iface_by_name(g, ifname))
return g;
}
return NULL;
}
struct fst_iface * fst_attach(const char *ifname, const u8 *own_addr, struct fst_iface * fst_attach(const char *ifname, const u8 *own_addr,
const struct fst_wpa_obj *iface_obj, const struct fst_wpa_obj *iface_obj,
const struct fst_iface_cfg *cfg) const struct fst_iface_cfg *cfg)
@ -60,14 +47,6 @@ struct fst_iface * fst_attach(const char *ifname, const u8 *own_addr,
WPA_ASSERT(iface_obj != NULL); WPA_ASSERT(iface_obj != NULL);
WPA_ASSERT(cfg != NULL); WPA_ASSERT(cfg != NULL);
g = fst_find_group_by_iface(ifname);
if (g) {
fst_printf(MSG_ERROR,
"%s: iface is already part of an FST group: %s",
ifname, g->group_id);
return NULL;
}
foreach_fst_group(g) { foreach_fst_group(g) {
if (os_strcmp(cfg->group_id, fst_group_get_id(g)) == 0) { if (os_strcmp(cfg->group_id, fst_group_get_id(g)) == 0) {
group = g; group = g;