> On 17 Oct 2016, at 10:35, Johannes Berg <johan...@sipsolutions.net> wrote:
>> On Mon, 2016-10-17 at 10:30 +0100, Ard Biesheuvel wrote:
>> Yes. But as I replied, setting the req size is not supported atm,
>> although it is reasonable to demand a way to allocate additional data
>> in the request specifically for this issue. So let's proceed with the
>> aead_request_alloc/free patch, but I would like to propose something
>> on the API side to address this particular issue
> Well, if your other patch to make it OK to be on-stack would be applied
> instead, this wouldn't make much sense either :)

Yes but that one only fixes ccm not gcm

> In this particular patch, we could reduce the size of the struct, but I
> don't actually think it'll make a difference to go from 48 to 36 bytes,
> given alignment etc., so I think I'll just leave it as is.

I understand you are in a hurry, but this is unlikely to be the only such 
issue. I will propose an 'auxdata' feature for the crypto api that can be used 
here, but also for any other occurrence where client data assoiciated with the 
request can no longer be allocated on the stack

Reply via email to