On Sun, Dec 06, 2015 at 09:47:33PM +0100, Michael Braun wrote: > This patch > 1. adds tagged vlan to struct vlan_description > (compile limited number of tagged vlans per description) > For k tagged vlans, the first k entries in vlan_description.tagged are > used. They are sorted in ascending order. All other entries are zero. > This way os_memcmp can find identical configurations. Is this os_memcmp use here referring to comparing memory presentations of C structures in full? If so, I don't really think that is a good approach as noted in the earlier comments in this thread. That is not safe if the compiler does any padding between the fields and some of those is not initialized and in any case, it would be much clearer to provide an explicit function to do such comparison of the exact struct members that need to be compared. -- Jouni Malinen PGP id EFC895FA _______________________________________________ Hostap mailing list Hostap@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/hostap
