[PATCH] Remove duplicated center_freq init

Jouni Malinen j at w1.fi
Thu Feb 18 07:58:21 PST 2016


On Tue, Feb 16, 2016 at 11:16:09AM +0100, eabinader at ocedo.com wrote:
> As the center freq values are adjusted in a proper
> function, dfs_adjust_vht_center_freq, just ensure
> there is no duplicated code for that.

Could you please clarify where these values are set in all possible code
paths? As far as I can tell, there are possible cases where it is far
from obvious that these get initialized. As such, I'd much rather leave
these here in dfs_get_valid_channel() to avoid getting uninitialized
values. It looks like much less work to do what is done currently rather
than go through each possible code path and add similar clearing to
places where it may be currently missing.
 
-- 
Jouni Malinen                                            PGP id EFC895FA



More information about the Hostap mailing list