* [PATCH] cfg80211: select CRYPTO_SHA256 if needed
@ 2017-11-24 8:35 Johannes Berg
2017-11-24 21:17 ` Florian Fainelli
0 siblings, 1 reply; 2+ messages in thread
From: Johannes Berg @ 2017-11-24 8:35 UTC (permalink / raw)
To: linux-wireless; +Cc: Florian Fainelli, Johannes Berg
From: Johannes Berg <johannes.berg@intel.com>
When regulatory database certificates are built-in, they're
currently using the SHA256 digest algorithm, so add that to
the build in that case.
Also add a note that for custom certificates, one may need
to add the right algorithms.
Reported-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
---
net/wireless/Kconfig | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/net/wireless/Kconfig b/net/wireless/Kconfig
index da91bb547db3..1abcc4fc4df1 100644
--- a/net/wireless/Kconfig
+++ b/net/wireless/Kconfig
@@ -20,6 +20,10 @@ config CFG80211
tristate "cfg80211 - wireless configuration API"
depends on RFKILL || !RFKILL
select FW_LOADER
+ # may need to update this when certificates are changed and are
+ # using a different algorithm, though right now they shouldn't
+ # (this is here rather than below to allow it to be a module)
+ select CRYPTO_SHA256 if CFG80211_USE_KERNEL_REGDB_KEYS
---help---
cfg80211 is the Linux wireless LAN (802.11) configuration API.
Enable this if you have a wireless device.
@@ -113,6 +117,9 @@ config CFG80211_EXTRA_REGDB_KEYDIR
certificates like in the kernel sources (net/wireless/certs/)
that shall be accepted for a signed regulatory database.
+ Note that you need to also select the correct CRYPTO_<hash> modules
+ for your certificates, and if cfg80211 is built-in they also must be.
+
config CFG80211_REG_CELLULAR_HINTS
bool "cfg80211 regulatory support for cellular base station hints"
depends on CFG80211_CERTIFICATION_ONUS
--
2.14.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] cfg80211: select CRYPTO_SHA256 if needed
2017-11-24 8:35 [PATCH] cfg80211: select CRYPTO_SHA256 if needed Johannes Berg
@ 2017-11-24 21:17 ` Florian Fainelli
0 siblings, 0 replies; 2+ messages in thread
From: Florian Fainelli @ 2017-11-24 21:17 UTC (permalink / raw)
To: Johannes Berg, linux-wireless; +Cc: Johannes Berg
On 11/24/2017 12:35 AM, Johannes Berg wrote:
> From: Johannes Berg <johannes.berg@intel.com>
>
> When regulatory database certificates are built-in, they're
> currently using the SHA256 digest algorithm, so add that to
> the build in that case.
>
> Also add a note that for custom certificates, one may need
> to add the right algorithms.
>
> Reported-by: Florian Fainelli <f.fainelli@gmail.com>
> Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Tested-by: Florian Fainelli <f.fainelli@gmail.com>
Thanks for the quick fix!
> ---
> net/wireless/Kconfig | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/net/wireless/Kconfig b/net/wireless/Kconfig
> index da91bb547db3..1abcc4fc4df1 100644
> --- a/net/wireless/Kconfig
> +++ b/net/wireless/Kconfig
> @@ -20,6 +20,10 @@ config CFG80211
> tristate "cfg80211 - wireless configuration API"
> depends on RFKILL || !RFKILL
> select FW_LOADER
> + # may need to update this when certificates are changed and are
> + # using a different algorithm, though right now they shouldn't
> + # (this is here rather than below to allow it to be a module)
> + select CRYPTO_SHA256 if CFG80211_USE_KERNEL_REGDB_KEYS
> ---help---
> cfg80211 is the Linux wireless LAN (802.11) configuration API.
> Enable this if you have a wireless device.
> @@ -113,6 +117,9 @@ config CFG80211_EXTRA_REGDB_KEYDIR
> certificates like in the kernel sources (net/wireless/certs/)
> that shall be accepted for a signed regulatory database.
>
> + Note that you need to also select the correct CRYPTO_<hash> modules
> + for your certificates, and if cfg80211 is built-in they also must be.
> +
> config CFG80211_REG_CELLULAR_HINTS
> bool "cfg80211 regulatory support for cellular base station hints"
> depends on CFG80211_CERTIFICATION_ONUS
>
--
Florian
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-11-24 21:17 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-24 8:35 [PATCH] cfg80211: select CRYPTO_SHA256 if needed Johannes Berg
2017-11-24 21:17 ` Florian Fainelli
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).