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!

Alan

Reply via email to