On Fri, Feb 3, 2017 at 12:09 PM, Jouni Malinen wrote: > On Mon, Jan 30, 2017 at 04:09:01PM +0000, Jose Blanquicet wrote: >> I tried what you suggested and it worked for me: >> >> diff --git a/src/drivers/driver_nl80211.c b/src/drivers/driver_nl80211.c >> index 6080079..5470312 100644 >> --- a/src/drivers/driver_nl80211.c >> +++ b/src/drivers/driver_nl80211.c >> @@ -3783,7 +3783,8 @@ static int wpa_driver_nl80211_set_ap(void *priv, >> beacon_set); >> if (beacon_set) >> cmd = NL80211_CMD_SET_BEACON; >> - else if (!nl80211_get_wiphy_data_ap(bss)) >> + else if (!drv->device_ap_sme && !drv->use_monitor && >> + !nl80211_get_wiphy_data_ap(bss)) >> return -ENOBUFS; >> >> wpa_hexdump(MSG_DEBUG, "nl80211: Beacon head", >> >> What does Jouni think about this solution? > > This looks like the correct thing to do here. > nl80211_get_wiphy_data_ap() used to be called from > nl80211_mgmt_subscribe_ap() and that function is within that same > !device_ap_sme && !use_monitor case. > > Could you please send this patch with a proper commit message and > Signed-off-by: line so that I can apply it? It would not be a problem at all but due to I just followed Lansky's suggestion, I was wondering if maybe he would prefer to do it himself. Lansky, would you? Regards, Jose Blanquicet _______________________________________________ Hostap mailing list Hostap@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/hostap
