Michael McConville wrote:
> Does this make sense?
I just realized that the allocation failure checks earlier in the
function return ENOBUFS. This probably makes more sense for the sake of
consistency.
> Index: sys/crypto/cryptosoft.c
> ===================================================================
> RCS file: /cvs/src/sys/crypto/cryptosoft.c,v
> retrieving revision 1.80
> diff -u -p -r1.80 cryptosoft.c
> --- sys/crypto/cryptosoft.c 10 Dec 2015 21:00:51 -0000 1.80
> +++ sys/crypto/cryptosoft.c 26 Feb 2016 17:21:00 -0000
> @@ -826,7 +826,7 @@ swcr_newsession(u_int32_t *sid, struct c
> M_CRYPTO_DATA, M_NOWAIT | M_ZERO);
> if ((*swd)->sw_kschedule == NULL) {
> swcr_freesession(i);
> - return EINVAL;
> + return ENOMEM;
> }
> }
> if (txf->setkey((*swd)->sw_kschedule, cri->cri_key,
>
