Re: [PATCHv2 01/12] Separate station grouping and uplink configuration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jan 07, 2016 at 02:22:33PM +0100, M. Braun wrote:

> >> diff --git a/src/ap/sta_info.h b/src/ap/sta_info.h
> >>  struct sta_info {
> >> +	struct hostapd_data *bss; /* required for set vlan in preauth */
> > 
> > Cannot say that I'm exactly fond of adding sta->bss pointer,
> 
> alternative approach: move get/set vlan for station out of pmksa2eapol
> and eapol2pmksa. That is struct eapol_state_machine would hold a struct
> *vlan_description instead of void *sta (to be filled during creation and
> updated by ap_sta_set_vlan, just referencing the struct sta_info
> member), so pmksa2eapol and eapol2pmksa can just copy the
> vlan_description data. ieee802_1x_new_station would then call
> ap_sta_set_vlan (using the eapol_state_machine member) in addition to
> the existing ap_sta_bind_vlan call afer pmksa2eapol, and
> ieee802_1x_new_station already has a hapd pointer available.

For now, I ended up removing this sta->bss pointer and passing the hapd
pointer to pmksa_cache_to_eapol_data(). pmksa_cache_auth.c is not really
supposed to use struct hapd_data, but at least this is now only for the
purpose of forwarding the pointer to ap_sta_set_vlan(). In fact, event
he use of struct sta_info in that function is not exactly in line with
the goal of keeping pmksa_cache_auth.c independent of the core hostapd
data structures. Anyway, that was not something added by this patch set.

-- 
Jouni Malinen                                            PGP id EFC895FA

_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap



[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux