Skip to content

Commit 8698baa

Browse files
ematsumiyasmfrench
authored andcommitted
smb3: rename encryption/decryption TFMs
Detach the TFM name from a specific algorithm (AES-CCM) as AES-GCM is also supported, making the name misleading. s/ccmaesencrypt/enc/ s/ccmaesdecrypt/dec/ Signed-off-by: Enzo Matsumiya <[email protected]> Reviewed-by: Paulo Alcantara (SUSE) <[email protected]> Signed-off-by: Steve French <[email protected]>
1 parent a4e430c commit 8698baa

File tree

4 files changed

+15
-16
lines changed

4 files changed

+15
-16
lines changed

fs/cifs/cifsencrypt.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -743,14 +743,14 @@ cifs_crypto_secmech_release(struct TCP_Server_Info *server)
743743
server->secmech.hmacmd5 = NULL;
744744
}
745745

746-
if (server->secmech.ccmaesencrypt) {
747-
crypto_free_aead(server->secmech.ccmaesencrypt);
748-
server->secmech.ccmaesencrypt = NULL;
746+
if (server->secmech.enc) {
747+
crypto_free_aead(server->secmech.enc);
748+
server->secmech.enc = NULL;
749749
}
750750

751-
if (server->secmech.ccmaesdecrypt) {
752-
crypto_free_aead(server->secmech.ccmaesdecrypt);
753-
server->secmech.ccmaesdecrypt = NULL;
751+
if (server->secmech.dec) {
752+
crypto_free_aead(server->secmech.dec);
753+
server->secmech.dec = NULL;
754754
}
755755

756756
kfree_sensitive(server->secmech.sdesccmacaes);

fs/cifs/cifsglob.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -171,8 +171,8 @@ struct cifs_secmech {
171171
struct sdesc *sdeschmacsha256; /* ctxt to generate smb2 signature */
172172
struct sdesc *sdesccmacaes; /* ctxt to generate smb3 signature */
173173
struct sdesc *sdescsha512; /* ctxt to generate smb3.11 signing key */
174-
struct crypto_aead *ccmaesencrypt; /* smb3 encryption aead */
175-
struct crypto_aead *ccmaesdecrypt; /* smb3 decryption aead */
174+
struct crypto_aead *enc; /* smb3 AEAD encryption TFM (AES-CCM and AES-GCM) */
175+
struct crypto_aead *dec; /* smb3 AEAD decryption TFM (AES-CCM and AES-GCM) */
176176
};
177177

178178
/* per smb session structure/fields */

fs/cifs/smb2ops.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4357,8 +4357,7 @@ crypt_message(struct TCP_Server_Info *server, int num_rqst,
43574357
return rc;
43584358
}
43594359

4360-
tfm = enc ? server->secmech.ccmaesencrypt :
4361-
server->secmech.ccmaesdecrypt;
4360+
tfm = enc ? server->secmech.enc : server->secmech.dec;
43624361

43634362
if ((server->cipher_type == SMB2_ENCRYPTION_AES256_CCM) ||
43644363
(server->cipher_type == SMB2_ENCRYPTION_AES256_GCM))

fs/cifs/smb2transport.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -904,7 +904,7 @@ smb3_crypto_aead_allocate(struct TCP_Server_Info *server)
904904
{
905905
struct crypto_aead *tfm;
906906

907-
if (!server->secmech.ccmaesencrypt) {
907+
if (!server->secmech.enc) {
908908
if ((server->cipher_type == SMB2_ENCRYPTION_AES128_GCM) ||
909909
(server->cipher_type == SMB2_ENCRYPTION_AES256_GCM))
910910
tfm = crypto_alloc_aead("gcm(aes)", 0, 0);
@@ -915,23 +915,23 @@ smb3_crypto_aead_allocate(struct TCP_Server_Info *server)
915915
__func__);
916916
return PTR_ERR(tfm);
917917
}
918-
server->secmech.ccmaesencrypt = tfm;
918+
server->secmech.enc = tfm;
919919
}
920920

921-
if (!server->secmech.ccmaesdecrypt) {
921+
if (!server->secmech.dec) {
922922
if ((server->cipher_type == SMB2_ENCRYPTION_AES128_GCM) ||
923923
(server->cipher_type == SMB2_ENCRYPTION_AES256_GCM))
924924
tfm = crypto_alloc_aead("gcm(aes)", 0, 0);
925925
else
926926
tfm = crypto_alloc_aead("ccm(aes)", 0, 0);
927927
if (IS_ERR(tfm)) {
928-
crypto_free_aead(server->secmech.ccmaesencrypt);
929-
server->secmech.ccmaesencrypt = NULL;
928+
crypto_free_aead(server->secmech.enc);
929+
server->secmech.enc = NULL;
930930
cifs_server_dbg(VFS, "%s: Failed to alloc decrypt aead\n",
931931
__func__);
932932
return PTR_ERR(tfm);
933933
}
934-
server->secmech.ccmaesdecrypt = tfm;
934+
server->secmech.dec = tfm;
935935
}
936936

937937
return 0;

0 commit comments

Comments
 (0)