[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-crypto-vger
Subject:    Re: [PATCH] fixed caam driver handling of sg list.
From:       Horia Geanta <horia.geanta () nxp ! com>
Date:       2019-01-31 11:20:59
Message-ID: VI1PR0402MB34858863410A59EFE18B8E0C98910 () VI1PR0402MB3485 ! eurprd04 ! prod ! outlook ! com
[Download RAW message or body]

On 1/31/2019 11:38 AM, Horia Geanta wrote:
> Thanks Pankaj.
> 
> On 1/31/2019 9:18 AM, Pankaj Gupta wrote:
>> when the source sg contains more than 1 fragment and
>> destination sg contains 1 fragment, the caam driver
>> mishandle the buffers to be sent to caam.
>>
> Patch subject prefix should be: "crypto: caam -"
> 
> Please Cc stable by adding:
> Cc: <stable@vger.kernel.org> # 4.3+
> 
> The root cause is commit:
> f2147b88b2b1 ("crypto: caam - Convert GCM to new AEAD interface")
> 
> however the issue was not visible until the switch to new AEAD interface
> completed, in commit:
> 479bcc7c5b9e ("crypto: caam - Convert authenc to new AEAD interface")
> 
> Thus it's probably correct to also add
> Fixes: 479bcc7c5b9e ("crypto: caam - Convert authenc to new AEAD interface")
> 
Actually for gcm the issue should show up from the very beginning, so
Fixes: f2147b88b2b1 ("crypto: caam - Convert GCM to new AEAD interface")
Cc: <stable@vger.kernel.org> # 4.2+
is better.

Thanks,
Horia
=
[prev in list] [next in list] [prev in thread] [next in thread]