1023975 - NPE when adding a card

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: Search Activity, defect)

Description

[Eric Edens [:eedens]]

W/dalvikvm(30488): threadid=1: thread exiting with uncaught exception (group=0x40c381f8)
E/AndroidRuntime(30488): FATAL EXCEPTION: main
E/AndroidRuntime(30488): java.lang.NullPointerException
E/AndroidRuntime(30488): 	at com.android.cardstream.CardStreamStateManager.showCard(CardStreamStateManager.java:212)
E/AndroidRuntime(30488): 	at com.mozilla.fennec.search.MainActivity.showCard(MainActivity.java:163)
E/AndroidRuntime(30488): 	at com.mozilla.fennec.search.MainActivity.addCard(MainActivity.java:158)
E/AndroidRuntime(30488): 	at com.mozilla.fennec.search.MainActivity.addCard(MainActivity.java:150)
E/AndroidRuntime(30488): 	at com.mozilla.fennec.search.agents.HttpAgent.onPostExecute(HttpAgent.java:154)
E/AndroidRuntime(30488): 	at android.os.AsyncTask.finish(AsyncTask.java:602)
E/AndroidRuntime(30488): 	at android.os.AsyncTask.access$600(AsyncTask.java:156)
E/AndroidRuntime(30488): 	at android.os.AsyncTask$InternalHandler.handleMessage(AsyncTask.java:615)
E/AndroidRuntime(30488): 	at android.os.Handler.dispatchMessage(Handler.java:99)
E/AndroidRuntime(30488): 	at android.os.Looper.loop(Looper.java:137)
E/AndroidRuntime(30488): 	at android.app.ActivityThread.main(ActivityThread.java:4507)
E/AndroidRuntime(30488): 	at java.lang.reflect.Method.invokeNative(Native Method)
E/AndroidRuntime(30488): 	at java.lang.reflect.Method.invoke(Method.java:511)
E/AndroidRuntime(30488): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:790)
E/AndroidRuntime(30488): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:557)
E/AndroidRuntime(30488): 	at dalvik.system.NativeStart.main(Native Method)
W/ActivityManager( 1992):   Force finishing activity r.intent.getComponent().flattenToShortString()
E/android.os.Debug( 1992): !@Dumpstate > dumpstate -k -t -n -z -d -o /data/log/dumpstate_app_error

Comment 1

[Eric Edens [:eedens]]

https://github.com/ericedens/FirefoxSearch/commit/9a2a9a35077d02e4bb1276f43864e8ebe19c8f70

Error came from a race condition between fragments being destroyed and AsyncTasks trying to update them.

The fix tracks the AsyncTasks that have been created. When the thread main thread is being stopped, it sends a cancel message to each task.

Comment 2

[Eric Edens [:eedens]]

This bug was filed against the prototype's codebase, which is no longer being used.  (https://github.com/ericedens/FirefoxSearch/tree/Prototype)