opentechinstitute / commotiond

The system management daemon for the Commotion Wireless Project.
https://commotionwireless.net
GNU Affero General Public License v3.0
26 stars 21 forks source link

commotiond/uci profile BSSID mismatch #69

Closed technosopher closed 10 years ago

technosopher commented 10 years ago

The output of commotion "Profile" bssid and the contents of /etc/commotion/profiles.d/Profile ("bssid": "02:CA:FF:EE:BA:BE") does not match the meshProfile's bssid value reported by /etc/config/ wireless (option bssid 'a6:a5:da:af:af:5f'). Additionally, I've confirmed that changing the channel and/or SSID of a node via the LuCI interface does NOT result in the regeneration of the bssid. The BSSID reported by both /etc/config/wireless and a third-party iwlist scan remains a6:a5:da:af:af:5f even after both SSID and channel are changed.

technosopher commented 10 years ago

Update: even the BSSID that appears to be dynamically-generated actually isn't being generated properly. It remains a6:a5:da:af:af:5f even when a non-default mesh SSID and channel is specified during the setup wizard

jheretic commented 10 years ago

I've identified two separate problems. One is that the code in the daemon generates an insufficiently unique BSSID, especially when it's the same SSID but different channels. That has now been fixed (pushing that code shortly). The second issue is that the logic in the /lib/wifi/mac80211.sh patch is not sufficiently rigorous when it comes to SSID and channel changes. That's relatively easy, but I won't be able to fix it until I can connect to a node, which I won't be able to do this evening.

jheretic commented 10 years ago

This was resolve in https://github.com/opentechinstitute/commotiond/commit/578ad38acf7cf61ad78b147bb4f24b41b98ffd14 and https://github.com/opentechinstitute/commotiond/commit/c08a6c52e23dded85f0b5da8ae7d2b822bc1bfdd