]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
wifi: lib80211: Constify struct lib80211_crypto_ops
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Mon, 5 Aug 2024 06:40:38 +0000 (08:40 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Tue, 27 Aug 2024 08:28:51 +0000 (10:28 +0200)
Now that functions in lib80211 handle "const struct lib80211_crypto_ops",
some structure can be constified as well.

Constifying these structures moves some data to a read-only section, so
increase overall security.

Before:
   text    data     bss     dec     hex filename
   7273     604      16    7893    1ed5 net/wireless/lib80211.o

After:
   text    data     bss     dec     hex filename
   7429     444      16    7889    1ed1 net/wireless/lib80211.o

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Reviewed-by: Simon Horman <horms@kernel.org>
Link: https://patch.msgid.link/0cc3741c15f2c502cc85bddda9d6582b5977c8f9.1722839425.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/wireless/lib80211.c
net/wireless/lib80211_crypt_ccmp.c
net/wireless/lib80211_crypt_tkip.c
net/wireless/lib80211_crypt_wep.c

index 51e31316bcb84da7a48b1cd1662cb98eaedf18bb..64c44704078611e033abfedce3802a653dc6d74a 100644 (file)
@@ -234,7 +234,7 @@ static void lib80211_crypt_null_deinit(void *priv)
 {
 }
 
-static struct lib80211_crypto_ops lib80211_crypt_null = {
+static const struct lib80211_crypto_ops lib80211_crypt_null = {
        .name = "NULL",
        .init = lib80211_crypt_null_init,
        .deinit = lib80211_crypt_null_deinit,
index cca5e1cf089ec0a3d25594925d5fc2d8520cbab4..5aad139130e1120a7f409de20c5252f694de3865 100644 (file)
@@ -418,7 +418,7 @@ static void lib80211_ccmp_print_stats(struct seq_file *m, void *priv)
                   ccmp->dot11RSNAStatsCCMPDecryptErrors);
 }
 
-static struct lib80211_crypto_ops lib80211_crypt_ccmp = {
+static const struct lib80211_crypto_ops lib80211_crypt_ccmp = {
        .name = "CCMP",
        .init = lib80211_ccmp_init,
        .deinit = lib80211_ccmp_deinit,
index 5c8cdf7681e35b883faf5b306abbe3027df11dd7..63e68e5e121e7357a76f0c5607d65716ab95b900 100644 (file)
@@ -705,7 +705,7 @@ static void lib80211_tkip_print_stats(struct seq_file *m, void *priv)
                   tkip->dot11RSNAStatsTKIPLocalMICFailures);
 }
 
-static struct lib80211_crypto_ops lib80211_crypt_tkip = {
+static const struct lib80211_crypto_ops lib80211_crypt_tkip = {
        .name = "TKIP",
        .init = lib80211_tkip_init,
        .deinit = lib80211_tkip_deinit,
index 6ab9957b8f969903820a52c374a876ff40b3c559..3b148c7bef850f1127867d6d58c0ce3258fb232c 100644 (file)
@@ -226,7 +226,7 @@ static void lib80211_wep_print_stats(struct seq_file *m, void *priv)
        seq_printf(m, "key[%d] alg=WEP len=%d\n", wep->key_idx, wep->key_len);
 }
 
-static struct lib80211_crypto_ops lib80211_crypt_wep = {
+static const struct lib80211_crypto_ops lib80211_crypt_wep = {
        .name = "WEP",
        .init = lib80211_wep_init,
        .deinit = lib80211_wep_deinit,