1884300 - Crash in [@ java.lang.IllegalArgumentException: at com.caverock.androidsvg.SVG.getDocumentViewBox(SVG.java)]

Status: VERIFIED FIXED

(Fenix :: General, defect, P1)

Description

[Ryan VanderMeulen [:RyanVM]]

Crash report: https://crash-stats.mozilla.org/report/index/ade41a46-8716-40b7-8da8-e94b10240308

Exception: java.lang.NullPointerException
Message: Attempt to read from field 'com.caverock.androidsvg.SVG$SvgContainer com.caverock.androidsvg.SVG$SvgObject.parent' on a null object reference in method 'void com.caverock.androidsvg.SVGParser.endElement(java.lang.String, java.lang.String, java.lang.String)'

Frame 	Module 									Function 		Source
0 	com.caverock.androidsvg.SVGParser 					endElement 		SVGParser.java:142
1 	com.caverock.androidsvg.SVGParser 					parseUsingXmlPullParser	SVGParser.java:130
2 	com.caverock.androidsvg.SVGParser 					parse 			SVGParser.java:56
3 	mozilla.components.browser.icons.decoder.SvgIconDecoder			decode 			SvgIconDecoder.kt:23
4 	mozilla.components.browser.icons.BrowserIconsKt 			decodeIconLoaderResult	BrowserIcons.kt:51
5 	mozilla.components.browser.icons.BrowserIconsKt 			access$load		BrowserIcons.kt:83
6 	mozilla.components.browser.icons.BrowserIcons$loadIconInternalAsync$1 	invokeSuspend		BrowserIcons.kt:93
7 	kotlin.coroutines.jvm.internal.BaseContinuationImpl 			resumeWith		ContinuationImpl.kt:9
8 	kotlinx.coroutines.DispatchedTask 					run			DispatchedTask.kt:111
9 	java.util.concurrent.ThreadPoolExecutor 				runWorker		ThreadPoolExecutor.java:1145
10 	java.util.concurrent.ThreadPoolExecutor$Worker 				run			ThreadPoolExecutor.java:644
11 	java.lang.Thread 							run			Thread.java:1012

Comment 1

[BMO Github Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1884300 - Add NullPointerException for SvgIconDecoder (#5948)

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The bug is marked as tracked for firefox125 (nightly). We have limited time to fix this, the soft freeze is in 6 days. However, the bug still isn't assigned.

:bclark, could you please find an assignee for this tracked bug? Given that it is a regression and we know the cause, we could also simply backout the regressor. If you disagree with the tracking decision, please talk with the release managers.

For more information, please visit BugBot documentation.

Comment 3

[BMO Github Automation]

Authored by https://github.com/jackyzy823
https://github.com/mozilla-mobile/firefox-android/commit/e8c1dac84db07ca6955b23ed794899c343dba1d8
[main] Bug 1884300 - Add NullPointerException for SvgIconDecoder