if (!bm->present)
return;
- if (bm == bst->primary_port);
+ if (bm == bst->primary_port)
bridge_reset_primary(bst);
if (bst->dev.active)
struct bridge_member *bm;
bm = calloc(1, sizeof(*bm) + strlen(dev->ifname) + 1);
+ if (!bm)
+ return NULL;
+
bm->bst = bst;
bm->dev.cb = bridge_member_cb;
bm->dev.hotplug = hotplug;
device_set_present(&bst->dev, true);
}
- if (!bst->ifnames)
- return;
-
vlist_update(&bst->members);
- blobmsg_for_each_attr(cur, bst->ifnames, rem) {
- bridge_add_member(bst, blobmsg_data(cur));
+ if (bst->ifnames) {
+ blobmsg_for_each_attr(cur, bst->ifnames, rem) {
+ bridge_add_member(bst, blobmsg_data(cur));
+ }
}
vlist_flush(&bst->members);
}