• Eric Biggers's avatar
    crypto: x86/aesni-gcm - fix crash on empty plaintext · 3af34963
    Eric Biggers authored
    gcmaes_crypt_by_sg() dereferences the NULL pointer returned by
    scatterwalk_ffwd() when encrypting an empty plaintext and the source
    scatterlist ends immediately after the associated data.
    
    Fix it by only fast-forwarding to the src/dst data scatterlists if the
    data length is nonzero.
    
    This bug is reproduced by the "rfc4543(gcm(aes))" test vectors when run
    with the new AEAD test manager.
    
    Fixes: e8455207 ("crypto: aesni - Update aesni-intel_glue to use scatter/gather")
    Cc: <stable@vger.kernel.org> # v4.17+
    Cc: Dave Watson <davejwatson@fb.com>
    Signed-off-by: default avatarEric Biggers <ebiggers@google.com>
    Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
    3af34963
aesni-intel_glue.c 38 KB