944635 - B2G RIL: Exception in updateDisplayCondition() ... "TypeError: RIL.operator is null"

Status: RESOLVED FIXED

(Firefox OS Graveyard :: RIL, defect)

Description

[Edgar Chen [:edgar]]

11-28 11:29:05.900 I/Gecko   (  106): RIL Worker[0]: Parcel handling threw TypeError: RIL.operator is null
11-28 11:29:05.900 I/Gecko   (  106): updateDisplayCondition@resource://gre/modules/ril_worker.js:12526
11-28 11:29:05.900 I/Gecko   (  106): callback@resource://gre/modules/ril_worker.js:11477
11-28 11:29:05.900 I/Gecko   (  106): processICCIOReadBinary@resource://gre/modules/ril_worker.js:11301
11-28 11:29:05.900 I/Gecko   (  106): ICC_COMMAND_READ_BINARY@resource://gre/modules/ril_worker.js:11335
11-28 11:29:05.900 I/Gecko   (  106): processICCIO@resource://gre/modules/ril_worker.js:11071
11-28 11:29:05.900 I/Gecko   (  106): REQUEST_SIM_IO@resource://gre/modules/ril_worker.js:5263
11-28 11:29:05.900 I/Gecko   (  106): handleParcel@resource://gre/modules/ril_worker.js:4885
11-28 11:29:05.900 I/Gecko   (  106): processParcel@resource://gre/modules/ril_worker.js:149
11-28 11:29:05.900 I/Gecko   (  106): processIncoming@blob:20620634-c668-4585-ac92-4d20b92f204e:536
11-28 11:29:05.900 I/Gecko   (  106): onRILMessage@resource://gre/modules/ril_worker.js:13555

Please see bug 944230 comment #2.

Comment 1

[Edgar Chen [:edgar]]

This bug was reported originally in bug 944230 comment #2.
And the log is located in attachment #8339707 [details].

Comment 2

[Hsin-Yi Tsai (she/her) [:hsinyi]]

See this issue on France Orange sim card, too.

Comment 3

[sam.hua]

happened on USIM card from CUCC.

Comment 4

[shawn ku [:sku]]

Attachment: Bug 944635 - B2G RIL: Exception in updateDisplayCondition() ... "TypeError: RIL.operator is null"

Comment 5

[Hsin-Yi Tsai (she/her) [:hsinyi]]

Comment on attachment 8443222 [details] [diff] [review]
Bug 944635 - B2G RIL: Exception in updateDisplayCondition() ... "TypeError: RIL.operator is null"

Review of attachment 8443222 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good.

Comment 6

[shawn ku [:sku]]

Attachment: Bug 944635 - B2G RIL: Exception in updateDisplayCondition() ... "TypeError: RIL.operator is null". r=HsinYi.

Comment 7

[shawn ku [:sku]]

try is green - https://tbpl.mozilla.org/?tree=Try&rev=67f008315f5e

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/b2g-inbound/rev/3a05fcd14cab

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/3a05fcd14cab

Comment 11

[Hsin-Yi Tsai (she/her) [:hsinyi]]

nominate 1.4? per Sam's comment in bug 1035058

Comment 12

[Preeti Raghunath(:Preeti)]

What's the user impact of not taking fix in 1.4? Sounds like confusing UX, is this right?

Comment 13

[shawn ku [:sku]]

(In reply to Preeti Raghunath(:Preeti) from comment #12)
> What's the user impact of not taking fix in 1.4? Sounds like confusing UX,
> is this right?

The impact will be wrong operator name displayed on UI.
Ex:
SPN: ABC
PLMN: 1234

Per 3GPP spec., We might need to display "ABC" as operator name for some circumstance, but show "1234" instead.

Comment 14

[Hsin-Yi Tsai (she/her) [:hsinyi]]

Thanks Shawn for the reply.

Comment 15

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/d5fb5bf53526
https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/875de32dc73a

Comment 16

[Sue]

Could you provide the detailed reproduce steps and video for me to verify this bug. Thanks!

Comment 17

[Edgar Chen [:edgar]]

Hi Sue, this bug is reproduced in booting up stage with the sim contains EF_SPN inserted.
The easier way to verify this is to check the log, after this fix you should not see any error like comment #0. Hope this help. Thank you.