Re: [PATCH v2] crypto: ctr - avoid VLA use
From: Salvatore Mesoraca
Date: Sat Mar 24 2018 - 09:22:38 EST
2018-03-23 16:36 GMT+01:00 Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>:
> On Thu, Mar 15, 2018 at 12:18:58PM +0100, Salvatore Mesoraca wrote:
>>
>> +#define MAX_BLOCKSIZE 16
>> +
>> +#ifdef CONFIG_HAVE_EFFICIENT_UNALIGNED_ACCESS
>> +#define MAX_ALIGNMASK 15
>> +#else
>> +#define MAX_ALIGNMASK 0
>> +#endif
>> +
>
> Hmm, this won't work. Just because you have efficient unaligned
> access in general doesn't mean that every implementation can live
> with unaligned access. In particular, on x86 there are quite a
> few implementations that require alignment or they will fault.
>
> So please just make it 15 unconditionally.
Oh, thank you for pointing it out. I'll fix this in v3.
Salvatore