Re: RFR 8164881: Add more tests for JDK-8139565.

2016-11-27 Thread Mallikarjuna Avaluri
Hi Artem Smotrakov, I have included the changes mentioned, please review the same. *Webrev: *http://cr.openjdk.java.net/~bgopularam/mavaluri/JDK-8164881/webrev.01/ * * Regards, Mallikarjuna Avaluri On 11/16/2016 4:1

Re: RFR 8170364: FilePermission path modified during merge

2016-11-27 Thread Wang Weijun
> On Nov 27, 2016, at 7:13 PM, Wang Weijun wrote: > >> >> On Nov 27, 2016, at 6:12 PM, Alan Bateman wrote: >> >> On 26/11/2016 08:54, Wang Weijun wrote: >> >>> Please take a review at >>> >>> http://cr.openjdk.java.net/~weijun/8170364/webrev.00/ >>> >>> The compatibility layer introduced

Re: Code Review Request JDK-8170329 New SSLSocket testing template

2016-11-27 Thread Xuelei Fan
On 11/27/2016 6:04 PM, Wang Weijun wrote: This is not only a test update. No, I happened to find an implementation issue with the new test, so fix it altogether. The issue is that the simple validator (SimpleValidator.java) does not support SKID/AKID during cert path build. If two trusted c

Re: RFR 8170364: FilePermission path modified during merge

2016-11-27 Thread Wang Weijun
> On Nov 27, 2016, at 6:12 PM, Alan Bateman wrote: > > On 26/11/2016 08:54, Wang Weijun wrote: > >> Please take a review at >> >>http://cr.openjdk.java.net/~weijun/8170364/webrev.00/ >> >> The compatibility layer introduced in the new FilePermission implementation >> requires one FilePer

Re: RFR 8170364: FilePermission path modified during merge

2016-11-27 Thread Alan Bateman
On 26/11/2016 08:54, Wang Weijun wrote: Please take a review at http://cr.openjdk.java.net/~weijun/8170364/webrev.00/ The compatibility layer introduced in the new FilePermission implementation requires one FilePermission to imply another with either a relative path or an absolute path. T

Re: Code Review Request JDK-8170329 New SSLSocket testing template

2016-11-27 Thread Wang Weijun
This is not only a test update. > On Nov 27, 2016, at 9:35 AM, Xuelei Fan wrote: > > Hi, > > Please review this test update: > > http://cr.openjdk.java.net/~xuelei/8170329/webrev.00/ > > The new template (SSLSocketTemplate.java) could be used to avoid the > anti-free-port issues. By using