> On Wed, Feb 17, 2021 at 12:14:33PM +0200, Andrei Otcheretianski wrote:
> > diff --git a/src/drivers/driver_nl80211.c
> > b/src/drivers/driver_nl80211.c
> > +static int
> > +send_and_recv_msgs_connect_handle(struct wpa_driver_nl80211_data
> *drv,
> > + struct nl_msg *msg, struct i802_bss *bss) {
> > + struct nl_sock *nl_connect = get_connect_handle(bss);
> > +
> > + if (nl_connect)
> > + return send_and_recv_msgs_owner(drv, msg, nl_connect, 1,
> > + process_bss_event, bss,
> NULL,
> > + NULL);
>
> So this has set_owner hardcoded to 1 which matches most modified callers,
> but...
>
> > @@ -10519,8 +10524,7 @@ static int
> wpa_driver_nl80211_leave_mesh(void
> > *priv)
> >
> > wpa_printf(MSG_DEBUG, "nl80211: mesh leave (ifindex=%d)", drv-
> >ifindex);
> > msg = nl80211_drv_msg(drv, 0, NL80211_CMD_LEAVE_MESH);
> > - ret = send_and_recv_msgs_owner(drv, msg,
> get_connect_handle(bss), 0,
> > - NULL, NULL, NULL, NULL);
> > + ret = send_and_recv_msgs_connect_handle(drv, msg, bss);
>
> This one here uses set_owner == 0.. Is it really correct to replace that with 1?
After looking at nl80211 code, looks that these attributes are ignored for leave_mesh().. So I guess this should be ok.
Andrei
>
> --
> Jouni Malinen PGP id EFC895FA
_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap
