Hi Fan, > > This patch fixes the out-of-bounds coverity issue by adding > missed algorithms to the array. > > Coverity issue: 337683 > > Fixes: c68d7aa354f6 ("crypto/aesni_mb: use architecture independent macros") > > Signed-off-by: Fan Zhang <roy.fan.zh...@intel.com> > --- > drivers/crypto/aesni_mb/rte_aesni_mb_pmd_private.h | 17 > ++++++++++++++++- > 1 file changed, 16 insertions(+), 1 deletion(-) > > diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_private.h > b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_private.h > index 4d439360f..dda78d989 100644 > --- a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_private.h > +++ b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_private.h > @@ -41,6 +41,14 @@ static const unsigned auth_blocksize[] = { > [SHA_512] = 128, > [AES_XCBC] = 16, > [AES_CCM] = 16, > + [AES_CMAC] = 16, > + [AES_GMAC] = 16, > + [AES_GCM] = 16, > + [PLAIN_SHA1] = 64, > + [PLAIN_SHA_224] = 64, > + [PLAIN_SHA_256] = 64, > + [PLAIN_SHA_384] = 128, > + [PLAIN_SHA_512] = 128 > }; > > /** > @@ -65,7 +73,13 @@ static const unsigned > auth_truncated_digest_byte_lengths[] = { > [AES_XCBC] = 12, > [AES_CMAC] = 12, > [AES_CCM] = 8, > - [NULL_HASH] = 0 > + [NULL_HASH] = 0, > + [AES_GMAC] = 16, > + [PLAIN_SHA1] = 20, > + [PLAIN_SHA_224] = 28, > + [PLAIN_SHA_256] = 32, > + [PLAIN_SHA_384] = 48, > + [PLAIN_SHA_512] = 64 > }; > > /** > @@ -90,6 +104,7 @@ static const unsigned auth_digest_byte_lengths[] = { > [SHA_512] = 64, > [AES_XCBC] = 16, > [AES_CMAC] = 16, > + [AES_CCM] = 16, > [AES_GMAC] = 12, > [NULL_HASH] = 0, > [PLAIN_SHA1] = 20, > -- > 2.14.5
Patchworks say that there is a compilation issue in this patch. Could you please check. Thanks, Akhil