• David McCullough's avatar
    [CRYPTO] aes: Fixed array boundary violation · 55e9dce3
    David McCullough authored
    The AES setkey routine writes 64 bytes to the E_KEY area even though
    there are only 60 bytes there.  It is in fact safe since E_KEY is
    immediately follwed by D_KEY which is initialised afterwards.  However,
    doing this may trigger undefined behaviour and makes Coverity unhappy.
    
    So by combining E_KEY and D_KEY into one array we sidestep this issue
    altogether.
    
    This problem was reported by Adrian Bunk.
    Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
    55e9dce3
aes.c 8.25 KB