Opened 5 weeks ago

Last modified 4 weeks ago

#2389 open defect

Android: AbstractWizardModel bug

Reported by: meeh Owned by: meeh
Priority: minor Milestone: 0.9.39
Component: apps/android Version: 0.9.37
Keywords: Cc:
Parent Tickets:

Description

java.lang.RuntimeException?:

at android.app.ActivityThread?.performLaunchActivity (ActivityThread?.java:2900)
at android.app.ActivityThread?.handleLaunchActivity (ActivityThread?.java:2979)
at android.app.ActivityThread?.handleRelaunchActivity (ActivityThread?.java:4868)
at android.app.ActivityThread?.-wrap18 (Unknown Source)
at android.app.ActivityThread?$H.handleMessage (ActivityThread?.java:1640)
at android.os.Handler.dispatchMessage (Handler.java:105)
at android.os.Looper.loop (Looper.java:164)
at android.app.ActivityThread?.main (ActivityThread?.java:6673)
at java.lang.reflect.Method.invoke (Native Method)
at com.android.internal.os.Zygote$MethodAndArgsCaller?.run (Zygote.java:240)
at com.android.internal.os.ZygoteInit?.main (ZygoteInit?.java:769)

Caused by: java.lang.NullPointerException?:

at net.i2p.android.wizard.model.AbstractWizardModel?.load (AbstractWizardModel?.java:70)
at net.i2p.android.wizard.ui.AbstractWizardActivity?.onCreate (AbstractWizardActivity?.java:52)
at android.app.Activity.performCreate (Activity.java:6984)
at android.app.Instrumentation.callActivityOnCreate (Instrumentation.java:1235)
at android.app.ActivityThread?.performLaunchActivity (ActivityThread?.java:2853)

Subtickets (add)

Change History (4)

comment:1 Changed 5 weeks ago by zzz

  • Milestone changed from eventually to 0.9.38
  • Status changed from new to open

I think you could fix this by making mRootPageList final.
Looks like mListeners and mContext could be final also.

comment:2 Changed 4 weeks ago by zab

I'm not familiar with the threading model on Android, but I suggest we make net.i2p.android.wizard.model.PageList extend CopyOnWriteArrayList

comment:3 Changed 4 weeks ago by meeh

Expanded the code and hope this happens again so we can figure out where the NPE came from.

comment:4 Changed 4 weeks ago by zzz

  • Milestone changed from 0.9.38 to 0.9.39

Due to a misunderstanding? My recommendation in comment 1 was not implemented for 38. Please implement it for .39.

Note: See TracTickets for help on using tickets.