Alan Maguire wrote: > Anurag Maskey wrote: >> code review request for >> >> 11297 nwam service must be restarted to clear out destroyed >> known-wlan entries >> http://defect.opensolaris.org/bz/show_bug.cgi?id=11297 >> >> webrev >> >> >> http://zhadum.east/export/ws/am223141/checkout-area/nwam1-fixes/webrev/ >> >> Known WLAN objects from nwamd have been removed. >> When a known WLAN is destroyed, any NCU that is connected to that >> wlan is taken down. When a known WLAN is created or edited, the scan >> thread is launched which connects to the best available wlan if not >> connected. >> > looks good. I was worried about the edge case > where we're connected to a known WLAN which > is a hidden WLAN and we delete the known WLAN > entry, but in this case I think nwamd_link_wifi_essid > records the hidden WLAN essid so the string comparison > in nwamd_ncu_known_wlan_destroyed() succeeds. > Nice work! Yep, the hidden WLAN's essid is copied to nwamd_link_wifi_essid in find_bssid_cb().
Anurag
