8143911: java/lang/StackWalker tests fail on Solaris with IllegalStateException

Mandy Chung mandy.chung at oracle.com
Tue Nov 24 21:08:33 UTC 2015


FWIW.  The entire webrev with this patch:
   http://cr.openjdk.java.net/~mchung/jdk9/jep259/webrev.05/

Mandy

> On Nov 24, 2015, at 12:54 PM, Mandy Chung <mandy.chung at oracle.com> wrote:
> 
> The failed tests now passed with fast debug and product build (manually verified on solaris x64 and sparcv9 machines).  JPRT job to run all core tests with fast debug and product build is in progress.
> 
> Mandy
> 
>> On Nov 24, 2015, at 12:52 PM, Mandy Chung <mandy.chung at oracle.com> wrote:
>> 
>> The value of anchor is an address and it’s an unsigned value.  The check should be fixed to handle unsigned value.  
>> 
>> diff --git a/src/java.base/share/classes/java/lang/StackStreamFactory.java b/src/java.base/share/classes/java/lang/StackStreamFactory.java
>> --- a/src/java.base/share/classes/java/lang/StackStreamFactory.java
>> +++ b/src/java.base/share/classes/java/lang/StackStreamFactory.java
>> @@ -230,8 +230,8 @@
>>                    }
>>                    break;
>>                case OPEN:
>> -                    if (this.anchor <= 0) {
>> -                        throw new IllegalStateException("This stack stream is not valid for walking");
>> +                    if (this.anchor == 0 || this.anchor == -1L) {
>> +                        throw new IllegalStateException("This stack stream is not valid for walking: " + anchor);
>>                    }
>>                    break;
>>                case CLOSED:
>> 
>> Thanks
>> Mandy
> 




More information about the core-libs-dev mailing list