Re: JDK 9 RFR of 4026465: Provide more byte array constructors for BigInteger

2015-01-02 Thread Alan Bateman
On 31/12/2014 02:15, joe darcy wrote: Hi Brian, The new changes generally look good. A few comments, for the new code like 291 } else if ((off < 0) || (off > val.length) || (len < 0) || 292((off + len) > val.length) || ((off + len) < 0)) { 293 thro

pre-RFR (s): 8049847: Enhance PrintWriter line.separator handling

2015-01-02 Thread Claes Redestad
Hi, this is a proposal to resolve concerns that PrintWriter/BufferedWriter behave inconsistently with j.u.Formatter when setting the line.separator property to override system-appropriate line breaks. Instead of providing a set of new constructors, I propose to simply add a new default met

RFR: 5049299 - (process) Use,posix_spawn, not fork, on S10 to avoid swap,exhaustion (jdk7u-dev)

2015-01-02 Thread Amit Baxi
Hi Folks, We are facing issue with process builder on JRockit JVM with solaris 5.10 machine. That is realted to following bug: http://bugs.java.com/view_bug.do?bug_id=5049299 Currently we are using JRockit JDK 1.6.0_37-R28.2.5-4.1.0 version. Please let me know following : *

Re: 5049299 - (process) Use,posix_spawn, not fork, on S10 to avoid swap,exhaustion (jdk7u-dev)

2015-01-02 Thread Rob McKenna
Hi Amit, As per https://bugs.openjdk.java.net/browse/JDK-5049299, this is fixed in 7u55+ (including 8 fcs). There are some peculiarities however: On JDK8 posix_spawn is the default on Solaris & Mac. vfork is the default on Linux. On JDK7u55+ posix_spawn is the default on Mac only. fork is the

Re: RFR: 5049299 - (process) Use,posix_spawn, not fork, on S10 to avoid swap,exhaustion (jdk7u-dev)

2015-01-02 Thread Volker Simonis
Hi Amit, I think this is the wrong place to ask. This list is about OpenJDK/OracleJDK. As you can read from the bug report (better look at: https://bugs.openjdk.java.net/browse/JDK-5049299) there's a list in which OpenJDK/OracleJDK versions this bug has been fixed. It describes that this bug hasn

Re: RFR: JDK-8051563: Convert JAXP function tests in xslt components: : org.apache.qetest.dtm package, org.apache.qetest.trax package to openjdk

2015-01-02 Thread huizhe wang
Lance, Tristan is looking into adding an extension base class for about 60 tests that require file permission, then the current base class would indeed set "minimal" permission. So please wait for his update :-) Best, Joe On 12/30/2014 3:07 PM, Lance @ Oracle wrote: Hi Tristan, I will look

Re: RFR: JDK-8051563: Convert JAXP function tests in xslt components: : org.apache.qetest.dtm package, org.apache.qetest.trax package to openjdk

2015-01-02 Thread Tristan Yan
Hi Joe and Lance Sorry for my late reply. I just uploaded the new webrev which is trying to limit minimal permissions for most of tests. The new changeset has two base classes; class JAXPBaseTest has only minimal set permissions; class JAXPFileBaseTest adds two permissions that need access local

Re: RFR 8066397 Remove network-related seed initialization code in ThreadLocal/SplittableRandom

2015-01-02 Thread Bradford Wetmore
the Microsoft C Runtime Library makes use of this function in its implementation of "rand_s", so it's removal would break a lot of programs. I think this is a relative guarantee that the function is here to stay. Ok. What are the fallbacks for SystemRandomImpl if /dev/urandom or the rtlGenRa

Re: Explicit Serialization API and Security

2015-01-02 Thread Brian Goetz
Overall the direction seems promising. Poking at it a bit... - ReadSerial methods are caller-sensitive and only show a class a view of its own fields. - Invariant checking is separate from deserialization, and does not seem entirely built-in -- subclass constructors seem responsible for ask

Re: RFR: JDK-8047769 SecureRandom should be more frugal with file descriptors

2015-01-02 Thread Bradford Wetmore
On 1/1/2015 12:22 PM, Peter Levart wrote: Hi Brad, Here's next webrev which tries to cover all your comments: http://cr.openjdk.java.net/~plevart/jdk9-dev/FileInputStreamPool.8047769/webrev.04/ I downloaded the webrev.05 (Chris' later followup email) and ran it through JPRT. The only error

Re: JDK 9 RFR of 4026465: Provide more byte array constructors for BigInteger

2015-01-02 Thread Brian Burkhalter
Hello all, Thanks for the comments. A new patch is here: http://cr.openjdk.java.net/~bpb/4026465/webrev.02/ On Dec 30, 2014, at 6:15 PM, joe darcy wrote: > The new changes generally look good. A few comments, for the new code like > > 291 } else if ((off < 0) || (off > val.length) ||