java.lang.IndexOutOfBoundsException: Index -1 out of bounds for length 1 (take 2) [375343407]

Fixed

Bug

Status Update

No update yet.

Description

si...@gmail.com

created issue #1

Oct 24, 2024 12:31PM

Component used: Activity

Versions used:

androidx.activity:activity-compose:1.9.3
androidx.activity:activity-ktx:1.9.3
androidx.navigation:navigation-compose:2.8.3

Devices/Android versions reproduced on: Pixel 7 API 35 (emulator)

Sample project: https://github.com/sindrenm/compose-navigation-crash-repro

This is a follow-up to 340202286, since #39 requested we open a new issue. I have a sample project (linked above) that reproduces the issue consistently by using the debugger, but we're also seeing this crash happen for our users (same stack trace) in production builds with the same library versions. Note that we have not been able to reproduce it locally without using the debugger, but our users are.

The stack trace is as follows:

FATAL EXCEPTION: main
Process: dev.sindrenm.repro.backnavigation, PID: 21646
java.lang.IndexOutOfBoundsException: Index -1 out of bounds for length 1
  at jdk.internal.util.Preconditions.outOfBounds(Preconditions.java:64)
  at jdk.internal.util.Preconditions.outOfBoundsCheckIndex(Preconditions.java:70)
  at jdk.internal.util.Preconditions.checkIndex(Preconditions.java:266)
  at java.util.Objects.checkIndex(Objects.java:359)
  at java.util.ArrayList.get(ArrayList.java:434)
  at androidx.navigation.compose.NavHostKt$NavHost$25$1.invokeSuspend(NavHost.kt:518)
  at androidx.navigation.compose.NavHostKt$NavHost$25$1.invoke(Unknown Source:8)
  at androidx.navigation.compose.NavHostKt$NavHost$25$1.invoke(Unknown Source:4)
  at androidx.activity.compose.OnBackInstance$job$1.invokeSuspend(PredictiveBackHandler.kt:121)
  at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
  at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:108)
  at androidx.compose.ui.platform.AndroidUiDispatcher.performTrampolineDispatch(AndroidUiDispatcher.android.kt:81)
  at androidx.compose.ui.platform.AndroidUiDispatcher.access$performTrampolineDispatch(AndroidUiDispatcher.android.kt:41)
  at androidx.compose.ui.platform.AndroidUiDispatcher$dispatchCallback$1.run(AndroidUiDispatcher.android.kt:57)
  at android.os.Handler.handleCallback(Handler.java:959)
  at android.os.Handler.dispatchMessage(Handler.java:100)
  at android.os.Looper.loopOnce(Looper.java:232)
  at android.os.Looper.loop(Looper.java:317)
  at android.app.ActivityThread.main(ActivityThread.java:8705)
  at java.lang.reflect.Method.invoke(Native Method)
  at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:580)
  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:886)

Comments

jo...@meterapp.co <jo...@meterapp.co> #2Oct 24, 2024 05:29PM

Project: platform/frameworks/support
Branch: androidx-main
Author: Clara Fok <clarafok@google.com>
Link: https://android-review.googlesource.com/3391123

Add non-reified NavGraphBuilder nested graph DSL

Expand for full commit details

Add non-reified NavGraphBuilder nested graph DSL 
 
Test: ./gradlew navigation:navigation-common:cC 
Bug: 382094941 
Relnote: "Added non-reified NavGraphBuilder.navigation extention functions to build a nested graph with route from KClass<T>" 
Change-Id: Ibf24453e66d69c00d4ffb3abe4dc5a1f3782ad09

Files:

M navigation/navigation-common/api/current.txt
M navigation/navigation-common/api/restricted_current.txt
M navigation/navigation-common/src/androidTest/java/androidx/navigation/NavGraphBuilderTest.kt
M navigation/navigation-common/src/main/java/androidx/navigation/NavGraphBuilder.kt

Hash: 94c09a993d68e5c57172c3e3bc418e35ec57f2fc
Date: Tue Dec 03 16:52:10 2024

jo...@meterapp.co <jo...@meterapp.co> #3Oct 24, 2024 05:30PM

Project: platform/frameworks/support
Branch: androidx-main
Author: Clara Fok <clarafok@google.com>
Link: https://android-review.googlesource.com/3390363

Add non-reified NavGraph.findNode safe args api

Expand for full commit details

Add non-reified NavGraph.findNode safe args api 
 
Test: ./gradlew navigation:navigation-common:cC 
Bug: 382094941 
Relnote: "Added non-reified NavGraph.findNode API to find node with KClass<*>" 
Change-Id: I54696ee5ce64305386de64654c6ce15f3a1fd741

Files:

M navigation/navigation-common/api/current.txt
M navigation/navigation-common/api/restricted_current.txt
M navigation/navigation-common/src/androidTest/java/androidx/navigation/NavGraphTest.kt
M navigation/navigation-common/src/main/java/androidx/navigation/NavGraph.kt

Hash: 3c80f14b77e8f2f84f961b24de924439fc009ad7
Date: Tue Dec 03 13:15:22 2024

bi...@gmail.com <bi...@gmail.com> #4Oct 24, 2024 09:40PM

Project: platform/frameworks/support
Branch: androidx-main
Author: Clara Fok <clarafok@google.com>
Link: https://android-review.googlesource.com/3390362

Add non-reified NavDeepLink.Builder.setUriPattern safe args api

Expand for full commit details

Add non-reified NavDeepLink.Builder.setUriPattern safe args api 
 
Test: existing tests 
Bug: 382094941 
Relnote: "Added non-reified NavDeepLink.Builder.setUriPattern API to add set deepLink Uri pattern with KClass<*>" 
Change-Id: Ia47be2a0ac9e2b8ef5def4aa808b939e15259d2a

Files:

M navigation/navigation-common/api/current.txt
M navigation/navigation-common/api/restricted_current.txt
M navigation/navigation-common/src/main/java/androidx/navigation/NavDeepLink.kt

Hash: 92ab76b0551be25fab92d2884a8c0c06a521af41
Date: Tue Dec 03 13:06:15 2024

[Deleted User] <[Deleted User]> #5Oct 29, 2024 02:09PM

Project: platform/frameworks/support
Branch: androidx-main
Author: Clara Fok <clarafok@google.com>
Link: https://android-review.googlesource.com/3390361

Add non-reified NavDestinationBuilder.deepLink safe args api

Expand for full commit details

Add non-reified NavDestinationBuilder.deepLink safe args api 
 
Test: ./gradlew navigation:navigation-common:cC 
Bug: 382094941 
Relnote: "Added non-reified NavDestinationBuilder.deepLink API to add a deeplink with KClass<*>" 
Change-Id: I93281ed88fe3be9e6c68a2d5f16d9e0f335c999d

Files:

M navigation/navigation-common/api/current.txt
M navigation/navigation-common/api/restricted_current.txt
M navigation/navigation-common/src/androidTest/java/androidx/navigation/NavDestinationBuilderTest.kt
M navigation/navigation-common/src/main/java/androidx/navigation/NavDestinationBuilder.kt

Hash: 3de22e912b5752790370c40acdc03de4d0985933
Date: Tue Dec 03 12:57:11 2024

8h...@gmail.com <8h...@gmail.com> #6Nov 5, 2024 01:08AM

Project: platform/frameworks/support
Branch: androidx-main
Author: Clara Fok <clarafok@google.com>
Link: https://android-review.googlesource.com/3390364

Add non-reified NavOptionsBuilder.popUpTo safe args api

Expand for full commit details

Add non-reified NavOptionsBuilder.popUpTo safe args api 
 
Test: ./gradlew navigation:navigation-common:cC 
Bug: 382094941 
Relnote: "Added non-reified NavOptionsBuilder.popUpTo API to set popUpTo destination with KClass<T>" 
Change-Id: Id09d2f511e9844b0618140781f07bebe1e7deb77

Files:

M navigation/navigation-common/api/current.txt
M navigation/navigation-common/api/restricted_current.txt
M navigation/navigation-common/src/androidTest/java/androidx/navigation/NavOptionsBuilderTest.kt
M navigation/navigation-common/src/main/java/androidx/navigation/NavOptionsBuilder.kt

Hash: e9d6539c2369ec0e0de4bccd18e890432cab86c2
Date: Tue Dec 03 14:33:41 2024

jo...@meterapp.co <jo...@meterapp.co> #7Nov 13, 2024 02:51PM

Project: platform/frameworks/support
Branch: androidx-main
Author: Clara Fok <clarafok@google.com>
Link: https://android-review.googlesource.com/3390365

Add non-reified NavOptions.Builder.setPopUpTo safe args api

Expand for full commit details

Add non-reified NavOptions.Builder.setPopUpTo safe args api 
 
Test: existing tests 
Bug: 382094941 
Relnote: "Added non-reified NavOptions.Builder.setPopUpTo API to set popUpTo destination with KClass<T>" 
Change-Id: I17d79479d8a1f2acda8f6767a40ff19773235d9d

Files:

M navigation/navigation-common/api/current.txt
M navigation/navigation-common/api/restricted_current.txt
M navigation/navigation-common/src/main/java/androidx/navigation/NavOptions.kt

Hash: 30bf6328f0dac6fc20aad820586cbe88c949cae2
Date: Tue Dec 03 14:44:33 2024

pa...@wewell.app <pa...@wewell.app> #8Nov 14, 2024 08:36AM

Project: platform/frameworks/support
Branch: androidx-main
Author: Clara Fok <clarafok@google.com>
Link: https://android-review.googlesource.com/3393181

Add non-reified NavDeepLinkDSLBuilder.navDeepLink safe args api

Expand for full commit details

Add non-reified NavDeepLinkDSLBuilder.navDeepLink safe args api 
 
Test: ./gradlew navigation:navigation-common:cC 
Bug: 382094941 
Relnote: "Added non-reified NavDeepLinkDSLBuilder.navDeepLink API to add a NavDeepLink with KClass<T>" 
Change-Id: Id28791416c67c0b4930370ded037ffb3c40922b6

Files:

M navigation/navigation-common/api/current.txt
M navigation/navigation-common/api/restricted_current.txt
M navigation/navigation-common/src/androidTest/java/androidx/navigation/NavDeepLinkBuilderTest.kt
M navigation/navigation-common/src/main/java/androidx/navigation/NavDeepLinkDslBuilder.kt
M navigation/navigation-common/src/main/java/androidx/navigation/NavDestinationBuilder.kt

Hash: 1f8d26c4db6329f6a726f317b2983cd5c7d8ea78
Date: Wed Dec 04 10:30:52 2024

jb...@google.com <jb...@google.com> Nov 15, 2024 07:43PM

Assigned to jb...@google.com.

ap...@google.com <ap...@google.com> #9Nov 20, 2024 02:06AM

Project: platform/frameworks/support
Branch: androidx-main
Author: Clara Fok <clarafok@google.com>
Link: https://android-review.googlesource.com/3393182

Add non-reified NavGraph.setStartDestination safe args api

Expand for full commit details

Add non-reified NavGraph.setStartDestination safe args api 
 
Test: ./gradlew navigation:navigation-common:cC 
Bug: 382094941 
Relnote: "Added non-reified NavGraph.setStartDestination API to set a graph’s startDestination with KClass<T>" 
Change-Id: Ia7eed5b9c0fe15834468d3eccc489c2dc3edca74

Files:

M navigation/navigation-common/api/current.txt
M navigation/navigation-common/api/restricted_current.txt
M navigation/navigation-common/src/androidTest/java/androidx/navigation/NavGraphTest.kt
M navigation/navigation-common/src/main/java/androidx/navigation/NavGraph.kt

Hash: 9e0a95788cbe0b0f44e1bafd1c93a086aced0ccb
Date: Wed Dec 04 10:40:39 2024

[Deleted User] <[Deleted User]> #10Nov 22, 2024 11:26AM

Fixed internally and available in navigation-2.9.0-alpha04

jb...@google.com <jb...@google.com> #11Nov 25, 2024 04:52PM

Marked as fixed.

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.navigation:navigation-common:2.9.0-alpha04

jo...@meterapp.co <jo...@meterapp.co> #12Nov 26, 2024 04:18PM

Thank you so much for your hard work and persistence on this issue! Really excited for the 2.8.5 release :)

sh...@navi.com <sh...@navi.com> #13Dec 3, 2024 07:50PM

by when can i expect 2.8.5 to hit stable and be available to use?

jo...@meterapp.co <jo...@meterapp.co> #14Dec 5, 2024 04:15PM

#comment13

by when can i expect 2.8.5 to hit stable and be available to use?

Based on the androidx release schedule, I'd assume it will be available December 11th. It looks like they skipped the thanksgiving release for most libraries and I'm hoping they'll be back with releases December 11th. https://developer.android.com/jetpack/androidx/versions/all-channel

pr...@google.com <pr...@google.com> #15Dec 11, 2024 06:27PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.navigation:navigation-compose:2.9.0-alpha04
androidx.navigation:navigation-compose:2.8.5

si...@gmail.com <si...@gmail.com> #16Dec 12, 2024 05:13PM

I can confirm that our usual way of reproducing this issue with the debugger no longer crashes. Thanks a lot for this release!

sa...@gmail.com <sa...@gmail.com> #17Dec 12, 2024 10:37PM

Works Well. Thank you.

ma...@wm.rs <ma...@wm.rs> #18Dec 16, 2024 12:53PM

Hi, I'm still facing the issue with 2.8.5

The stack trace:

Fatal Exception: java.lang.IndexOutOfBoundsException: Index -1 out of bounds for length 1
       at jdk.internal.util.Preconditions.outOfBounds(Preconditions.java:64)
       at jdk.internal.util.Preconditions.outOfBoundsCheckIndex(Preconditions.java:70)
       at jdk.internal.util.Preconditions.checkIndex(Preconditions.java:266)
       at java.util.Objects.checkIndex(Objects.java:359)
       at java.util.ArrayList.get(ArrayList.java:434)
       at androidx.navigation.compose.NavHostKt$NavHost$28$1.invokeSuspend(NavHost.kt:620)
       at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
       at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:101)
       at androidx.compose.ui.platform.AndroidUiDispatcher.performTrampolineDispatch(AndroidUiDispatcher.android.kt:81)
       at androidx.compose.ui.platform.AndroidUiDispatcher.access$performTrampolineDispatch(AndroidUiDispatcher.android.kt:41)
       at androidx.compose.ui.platform.AndroidUiDispatcher$dispatchCallback$1.run(AndroidUiDispatcher.android.kt:57)
       at android.os.Handler.handleCallback(Handler.java:958)
       at android.os.Handler.dispatchMessage(Handler.java:99)
       at android.os.Looper.loopOnce(Looper.java:230)
       at android.os.Looper.loop(Looper.java:319)
       at android.app.ActivityThread.main(ActivityThread.java:9063)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:588)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1103)

It's seems the issue now is in val previousEntry = currentBackStack[currentBackStack.size - 2]:

if (inPredictiveBack) {
    LaunchedEffect(progress) {
        val previousEntry = currentBackStack[currentBackStack.size - 2]
        transitionState.seekTo(progress, previousEntry)
    }
}

si...@gmail.com <si...@gmail.com> #19Dec 16, 2024 04:10PM

Filed https://issuetracker.google.com/issues/384186542 for the new issue.

ba...@gmail.com <ba...@gmail.com> #20Dec 16, 2024 07:07PM

Version 2.8.5 resolved the issue. However, this time instead of throwing an exception, it neither navigates up nor navigates. No solution.

Issue 375343407

Description

Issue summary

Comments

jo...@meterapp.co <jo...@meterapp.co> #2Oct 24, 2024 05:29PM

jo...@meterapp.co <jo...@meterapp.co> #3Oct 24, 2024 05:30PM

bi...@gmail.com <bi...@gmail.com> #4Oct 24, 2024 09:40PM

[Deleted User] <[Deleted User]> #5Oct 29, 2024 02:09PM

8h...@gmail.com <8h...@gmail.com> #6Nov 5, 2024 01:08AM

jo...@meterapp.co <jo...@meterapp.co> #7Nov 13, 2024 02:51PM

pa...@wewell.app <pa...@wewell.app> #8Nov 14, 2024 08:36AM

jb...@google.com <jb...@google.com> Nov 15, 2024 07:43PM

ap...@google.com <ap...@google.com> #9Nov 20, 2024 02:06AM

[Deleted User] <[Deleted User]> #10Nov 22, 2024 11:26AM

jb...@google.com <jb...@google.com> #11Nov 25, 2024 04:52PM

jo...@meterapp.co <jo...@meterapp.co> #12Nov 26, 2024 04:18PM

sh...@navi.com <sh...@navi.com> #13Dec 3, 2024 07:50PM

jo...@meterapp.co <jo...@meterapp.co> #14Dec 5, 2024 04:15PM

pr...@google.com <pr...@google.com> #15Dec 11, 2024 06:27PM

si...@gmail.com <si...@gmail.com> #16Dec 12, 2024 05:13PM

sa...@gmail.com <sa...@gmail.com> #17Dec 12, 2024 10:37PM

ma...@wm.rs <ma...@wm.rs> #18Dec 16, 2024 12:53PM

si...@gmail.com <si...@gmail.com> #19Dec 16, 2024 04:10PM

ba...@gmail.com <ba...@gmail.com> #20Dec 16, 2024 07:07PM

Add comment

Issue metadata