On Tue, Apr 15, 2014 at 11:23 AM, Dien Nguyen
<nguyentienhoangd...@gmail.com> wrote:

> The ICV failed because the scatterlist dst_sg is created with shorter length
> than required (should be caop->len +
> cryptodev_cipher_get_tag_size(&ses_ptr->cdata)).

Hello Dien,
 Your analysis seem correct.

> A possible fix is to change
> rc = __get_userbuf(src, src_len, 1, ses->used_pages,
>                                        ses->pages, ses->sg, task, mm);
> to
> rc = __get_userbuf(dst, dst_len, 1, ses->used_pages,
>                                        ses->pages, ses->sg, task, mm);

That looks ok. What I'm afraid is whether it could affect some other
places in code, although I cannot think any valid case where dst_len
could be smaller than src_len.

regards,
Nikos

_______________________________________________
Cryptodev-linux-devel mailing list
Cryptodev-linux-devel@gna.org
https://mail.gna.org/listinfo/cryptodev-linux-devel

Reply via email to