Skip to content

Commit 75e1f66

Browse files
committed
Merge tag 'libcrypto-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux
Pull crypto library fix from Eric Biggers: "Fix a big endian specific issue in the PPC64-optimized AES code" * tag 'libcrypto-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/linux: lib/crypto: powerpc/aes: Fix rndkey_from_vsx() on big endian CPUs
2 parents aaf96df + beeebff commit 75e1f66

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

lib/crypto/powerpc/aes.h

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,8 @@ static inline bool is_vsx_format(const struct p8_aes_key *key)
9595
}
9696

9797
/*
98-
* Convert a round key from VSX to generic format by reflecting the 16 bytes,
98+
* Convert a round key from VSX to generic format by reflecting all 16 bytes (if
99+
* little endian) or reflecting the bytes in each 4-byte word (if big endian),
99100
* and (if apply_inv_mix=true) applying InvMixColumn to each column.
100101
*
101102
* It would be nice if the VSX and generic key formats would be compatible. But
@@ -107,6 +108,7 @@ static inline bool is_vsx_format(const struct p8_aes_key *key)
107108
*/
108109
static void rndkey_from_vsx(u32 out[4], const u32 in[4], bool apply_inv_mix)
109110
{
111+
const bool be = IS_ENABLED(CONFIG_CPU_BIG_ENDIAN);
110112
u32 k0 = swab32(in[0]);
111113
u32 k1 = swab32(in[1]);
112114
u32 k2 = swab32(in[2]);
@@ -118,10 +120,10 @@ static void rndkey_from_vsx(u32 out[4], const u32 in[4], bool apply_inv_mix)
118120
k2 = inv_mix_columns(k2);
119121
k3 = inv_mix_columns(k3);
120122
}
121-
out[0] = k3;
122-
out[1] = k2;
123-
out[2] = k1;
124-
out[3] = k0;
123+
out[0] = be ? k0 : k3;
124+
out[1] = be ? k1 : k2;
125+
out[2] = be ? k2 : k1;
126+
out[3] = be ? k3 : k0;
125127
}
126128

127129
static void aes_preparekey_arch(union aes_enckey_arch *k,

0 commit comments

Comments
 (0)