summaryrefslogtreecommitdiffstats
path: root/net/wireless
diff options
context:
space:
mode:
authorEric Biggers <ebiggers@google.com>2018-11-14 15:19:39 -0500
committerHerbert Xu <herbert@gondor.apana.org.au>2018-11-20 01:26:55 -0500
commit1ad0f1603a6b2afb62a1c065409aaa4e43ca7627 (patch)
treedbc2b8fbc9998f36e8d079c585da7786d53eeda0 /net/wireless
parentd41655909e3236bfb00aa69f435a9634cd74b60b (diff)
crypto: drop mask=CRYPTO_ALG_ASYNC from 'cipher' tfm allocations
'cipher' algorithms (single block ciphers) are always synchronous, so passing CRYPTO_ALG_ASYNC in the mask to crypto_alloc_cipher() has no effect. Many users therefore already don't pass it, but some still do. This inconsistency can cause confusion, especially since the way the 'mask' argument works is somewhat counterintuitive. Thus, just remove the unneeded CRYPTO_ALG_ASYNC flags. This patch shouldn't change any actual behavior. Signed-off-by: Eric Biggers <ebiggers@google.com> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'net/wireless')
-rw-r--r--net/wireless/lib80211_crypt_ccmp.c2
-rw-r--r--net/wireless/lib80211_crypt_tkip.c4
-rw-r--r--net/wireless/lib80211_crypt_wep.c4
3 files changed, 5 insertions, 5 deletions
diff --git a/net/wireless/lib80211_crypt_ccmp.c b/net/wireless/lib80211_crypt_ccmp.c
index 6beab0cfcb99..55214fe925b2 100644
--- a/net/wireless/lib80211_crypt_ccmp.c
+++ b/net/wireless/lib80211_crypt_ccmp.c
@@ -75,7 +75,7 @@ static void *lib80211_ccmp_init(int key_idx)
75 goto fail; 75 goto fail;
76 priv->key_idx = key_idx; 76 priv->key_idx = key_idx;
77 77
78 priv->tfm = crypto_alloc_cipher("aes", 0, CRYPTO_ALG_ASYNC); 78 priv->tfm = crypto_alloc_cipher("aes", 0, 0);
79 if (IS_ERR(priv->tfm)) { 79 if (IS_ERR(priv->tfm)) {
80 priv->tfm = NULL; 80 priv->tfm = NULL;
81 goto fail; 81 goto fail;
diff --git a/net/wireless/lib80211_crypt_tkip.c b/net/wireless/lib80211_crypt_tkip.c
index b5e235573c8a..35f06563207d 100644
--- a/net/wireless/lib80211_crypt_tkip.c
+++ b/net/wireless/lib80211_crypt_tkip.c
@@ -99,7 +99,7 @@ static void *lib80211_tkip_init(int key_idx)
99 99
100 priv->key_idx = key_idx; 100 priv->key_idx = key_idx;
101 101
102 priv->tx_tfm_arc4 = crypto_alloc_cipher("arc4", 0, CRYPTO_ALG_ASYNC); 102 priv->tx_tfm_arc4 = crypto_alloc_cipher("arc4", 0, 0);
103 if (IS_ERR(priv->tx_tfm_arc4)) { 103 if (IS_ERR(priv->tx_tfm_arc4)) {
104 priv->tx_tfm_arc4 = NULL; 104 priv->tx_tfm_arc4 = NULL;
105 goto fail; 105 goto fail;
@@ -111,7 +111,7 @@ static void *lib80211_tkip_init(int key_idx)
111 goto fail; 111 goto fail;
112 } 112 }
113 113
114 priv->rx_tfm_arc4 = crypto_alloc_cipher("arc4", 0, CRYPTO_ALG_ASYNC); 114 priv->rx_tfm_arc4 = crypto_alloc_cipher("arc4", 0, 0);
115 if (IS_ERR(priv->rx_tfm_arc4)) { 115 if (IS_ERR(priv->rx_tfm_arc4)) {
116 priv->rx_tfm_arc4 = NULL; 116 priv->rx_tfm_arc4 = NULL;
117 goto fail; 117 goto fail;
diff --git a/net/wireless/lib80211_crypt_wep.c b/net/wireless/lib80211_crypt_wep.c
index 6015f6b542a6..20c1ad63ad44 100644
--- a/net/wireless/lib80211_crypt_wep.c
+++ b/net/wireless/lib80211_crypt_wep.c
@@ -48,13 +48,13 @@ static void *lib80211_wep_init(int keyidx)
48 goto fail; 48 goto fail;
49 priv->key_idx = keyidx; 49 priv->key_idx = keyidx;
50 50
51 priv->tx_tfm = crypto_alloc_cipher("arc4", 0, CRYPTO_ALG_ASYNC); 51 priv->tx_tfm = crypto_alloc_cipher("arc4", 0, 0);
52 if (IS_ERR(priv->tx_tfm)) { 52 if (IS_ERR(priv->tx_tfm)) {
53 priv->tx_tfm = NULL; 53 priv->tx_tfm = NULL;
54 goto fail; 54 goto fail;
55 } 55 }
56 56
57 priv->rx_tfm = crypto_alloc_cipher("arc4", 0, CRYPTO_ALG_ASYNC); 57 priv->rx_tfm = crypto_alloc_cipher("arc4", 0, 0);
58 if (IS_ERR(priv->rx_tfm)) { 58 if (IS_ERR(priv->rx_tfm)) {
59 priv->rx_tfm = NULL; 59 priv->rx_tfm = NULL;
60 goto fail; 60 goto fail;