Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[android] Fixes #10563 NPE addAnnotationIcon #10804

Closed
wants to merge 1 commit into from
Closed

[android] Fixes #10563 NPE addAnnotationIcon #10804

wants to merge 1 commit into from

Conversation

hashfunction
Copy link

Full trace is below. This was captured with 5.2.1 release. It would be good to pull this in any upcoming releases as this is a high rate crash amongst our users.

Fatal Exception: java.lang.NullPointerException: Attempt to invoke virtual method 'void com.mapbox.mapboxsdk.maps.NativeMapView.addAnnotationIcon(java.lang.String, int, int, float, byte[])' on a null object reference
at com.mapbox.mapboxsdk.maps.IconManager.loadIcon(SourceFile:115)
at com.mapbox.mapboxsdk.maps.IconManager.(SourceFile)
at com.mapbox.mapboxsdk.maps.MapView.initialiseMap(SourceFile:174)
at com.mapbox.mapboxsdk.maps.MapView.access$800(SourceFile:70)
at com.mapbox.mapboxsdk.maps.MapView$6$1.run(SourceFile:326)
at android.os.Handler.handleCallback(Handler.java:739)
at android.os.Handler.dispatchMessage(Handler.java:95)
at android.os.Looper.loop(Looper.java:135)
at android.app.ActivityThread.main(ActivityThread.java:5491)
at java.lang.reflect.Method.invoke(Method.java)
at java.lang.reflect.Method.invoke(Method.java:372)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:984)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:779)

@tobrun
Copy link
Member

tobrun commented Jan 2, 2018

Thank you for the contribution @hashfunction, I'm pulling your changes into #10811 to follow up on this.

@tobrun tobrun closed this Jan 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants