8143911: java/lang/StackWalker tests fail on Solaris with IllegalStateException
Mandy Chung
mandy.chung at oracle.com
Tue Nov 24 20:54:46 UTC 2015
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