[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-core-libs-dev
Subject:    Re: [jdk17] RFR: 8269034: AccessControlException for SunPKCS11 daemon threads [v3]
From:       Valerie Peng <valeriep () openjdk ! java ! net>
Date:       2021-06-29 17:47:07
Message-ID: H5gHVQ80o6twpL11OOlunxXSZF2Ht5bqpAC6MTK-nUU=.93172c09-9b4c-4f8a-892d-29f3f4264d4b () github ! com
[Download RAW message or body]

On Tue, 29 Jun 2021 00:07:41 GMT, Sean Coffey <coffeys@openjdk.org> wrote:

> > Sufficient permissions missing if this code was ever to run with SecurityManager. \
> >  
> > Cleanest approach appears to be use of InnocuousThread to create the \
> > cleaner/poller threads. Test case coverage extended to cover the SecurityManager \
> > scenario. 
> > Reviewer request: @valeriepeng
> 
> Sean Coffey has updated the pull request with a new target base due to a merge or a \
> rebase. The incremental webrev excludes the unrelated changes brought in by the \
> merge/rebase. The pull request contains four additional commits since the last \
> revision: 
> - Edits from review
> - Merge remote-tracking branch 'origin/master' into pkcs11-perms
> - Move TokenPoller to Runnable
> - 8269034: AccessControlException for SunPKCS11 daemon threads

Update looks good. Thanks, Valerie

-------------

Marked as reviewed by valeriep (Reviewer).

PR: https://git.openjdk.java.net/jdk17/pull/117


[prev in list] [next in list] [prev in thread] [next in thread]