[illumos-Developer] [REVIEW] 289 invalid padding when using java pkcs11 provider

Jason King jason.brian.king at gmail.com
Fri Oct 8 11:35:34 PDT 2010


I believe Rich said that the code is (currently) at least never used
in the kernel (which is probably why that KM_SLEEP never caused a
compile failure), but yeah, might as well do it right while we're
working on it.

Would it be advisable to put the final if (s != NULL) under an
#if..#endif for any editors as well?:

    #ifdef _KERNEL
    if (s != NULL)
        kmem_free(s, s_size);
    #else
    if (s != NULL)
        free(s);
    #endif

On Fri, Oct 8, 2010 at 1:11 PM, Gordon Ross <gordon.w.ross at gmail.com> wrote:
> Oh, and the kmem_alloc needs another arg, I think :)
> i.e. KM_SLEEP
>



More information about the Developer mailing list