]> git.baikalelectronics.ru Git - kernel.git/commit
crypto: aesni - fix "by8" variant for 128 bit keys
authorMathias Krause <minipli@googlemail.com>
Tue, 30 Dec 2014 21:50:54 +0000 (22:50 +0100)
committerHerbert Xu <herbert@gondor.apana.org.au>
Mon, 5 Jan 2015 10:35:02 +0000 (21:35 +1100)
commitb4e9bc4b7f180dd659a5b11c1c89606da1ba286d
tree3494012be5e6e4911c9069e1dcaccdf2203ab2f9
parentf606189e0a4ad705acbe050a887803692b573daa
crypto: aesni - fix "by8" variant for 128 bit keys

The "by8" counter mode optimization is broken for 128 bit keys with
input data longer than 128 bytes. It uses the wrong key material for
en- and decryption.

The key registers xkey0, xkey4, xkey8 and xkey12 need to be preserved
in case we're handling more than 128 bytes of input data -- they won't
get reloaded after the initial load. They must therefore be (a) loaded
on the first iteration and (b) be preserved for the latter ones. The
implementation for 128 bit keys does not comply with (a) nor (b).

Fix this by bringing the implementation back to its original source
and correctly load the key registers and preserve their values by
*not* re-using the registers for other purposes.

Kudos to James for reporting the issue and providing a test case
showing the discrepancies.

Reported-by: James Yonan <james@openvpn.net>
Cc: Chandramouli Narayanan <mouli@linux.intel.com>
Cc: <stable@vger.kernel.org> # v3.18
Signed-off-by: Mathias Krause <minipli@googlemail.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
arch/x86/crypto/aes_ctrby8_avx-x86_64.S