Skip to content

Commit

Permalink
cfg80211: initialize regulatory keys/database later
Browse files Browse the repository at this point in the history
When cfg80211 is built as a module, everything is fine, and we
can keep the code as is; in fact, we have to, because there can
only be a single module_init().

When cfg80211 is built-in, however, it needs to initialize
before drivers (device_initcall/module_init), and thus used to
be at subsys_initcall(). I'd moved it to fs_initcall() earlier,
where it can remain. However, this is still too early because at
that point the key infrastructure hasn't been initialized yet,
so X.509 certificates can't be parsed yet.

To work around this problem, load the regdb keys only later in
a late_initcall(), at which point the necessary infrastructure
has been initialized.

Fixes: 90a53e4 ("cfg80211: implement regdb signature checking")
Reported-by: Xiaolong Ye <xiaolong.ye@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
  • Loading branch information
Johannes Berg committed Nov 20, 2017
1 parent 7cca2ac commit d7be102
Showing 1 changed file with 27 additions and 15 deletions.
42 changes: 27 additions & 15 deletions net/wireless/reg.c
Original file line number Diff line number Diff line change
Expand Up @@ -3644,27 +3644,14 @@ void regulatory_propagate_dfs_state(struct wiphy *wiphy,
}
}

int __init regulatory_init(void)
static int __init regulatory_init_db(void)
{
int err = 0;
int err;

err = load_builtin_regdb_keys();
if (err)
return err;

reg_pdev = platform_device_register_simple("regulatory", 0, NULL, 0);
if (IS_ERR(reg_pdev))
return PTR_ERR(reg_pdev);

spin_lock_init(&reg_requests_lock);
spin_lock_init(&reg_pending_beacons_lock);
spin_lock_init(&reg_indoor_lock);

rcu_assign_pointer(cfg80211_regdomain, cfg80211_world_regdom);

user_alpha2[0] = '9';
user_alpha2[1] = '7';

/* We always try to get an update for the static regdomain */
err = regulatory_hint_core(cfg80211_world_regdom->alpha2);
if (err) {
Expand Down Expand Up @@ -3692,6 +3679,31 @@ int __init regulatory_init(void)

return 0;
}
#ifndef MODULE
late_initcall(regulatory_init_db);
#endif

int __init regulatory_init(void)
{
reg_pdev = platform_device_register_simple("regulatory", 0, NULL, 0);
if (IS_ERR(reg_pdev))
return PTR_ERR(reg_pdev);

spin_lock_init(&reg_requests_lock);
spin_lock_init(&reg_pending_beacons_lock);
spin_lock_init(&reg_indoor_lock);

rcu_assign_pointer(cfg80211_regdomain, cfg80211_world_regdom);

user_alpha2[0] = '9';
user_alpha2[1] = '7';

#ifdef MODULE
return regulatory_init_db();
#else
return 0;
#endif
}

void regulatory_exit(void)
{
Expand Down

0 comments on commit d7be102

Please sign in to comment.