Automatic Roaming Broker 3G does not work correctly

NEW
Unassigned

Status

Firefox OS
Gaia::Settings
3 years ago
3 years ago

People

(Reporter: Juanjo Iglesias, Unassigned)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

3 years ago
Automatic Roaming Broker 3G should register the device in the network without asking for PIN code, but the device asks for the PIN code again.

I have tested it with flame master (Gecko 0e0825b Gaia dedd8c6) and flame 2.0 (Gecko 1f9f165 Gaia 86d83f4)

STEP
- Insert the Roaming Broker 3G SIM card (This is a special SIM card)
- Enter the PIN code
- Wait until the DUT register in the network.


ACTUAL
The device asks for the PIN code again before registration.

EXPECTED
The device should registers in the network without ask for for the PIN code again. 


ADDITIONAL: 
To test this issue you need a Roaming Broker 3G SIM Card.
(Reporter)

Comment 1

3 years ago
Created attachment 8511900 [details]
log_Broker3G_master.txt
Component: Gaia → Gaia::Settings
Created attachment 8514842 [details] [review]
Proposed patch

I saw this into the traces:

I/GeckoDump(  206): [system] STK Proactive Command bad formed: {"iccId":"8934072100187087831","command":{"commandNumber":1,"typeOfCommand":1,"commandQualifier":6,"rilMessageType":"stkcommand","options":null,"rilMessageClientId":0}}

This is because we check for options here [1]

The proposed patch checks for options existance but accepts null value.

Juanjo, Can you test again with the patch? I don't have any SIM with roaming broker support. Thanks in advance.

[1] https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/icc.js#L133
Attachment #8514842 - Flags: feedback?(juanjose.iglesias)
(Reporter)

Comment 3

3 years ago
I have taken a new log using a flame master (Gecko c0265bd Gaia 0bcc566), with the patch added, and a Broker 3G SIM card.

I activated gaia traces and ril debug to take it. 

During the registration in the network, the device asks for the PIN and should not do .

The new log is called Broker3G_RilOn_TracesOn_master.txt
(Reporter)

Comment 4

3 years ago
Created attachment 8516017 [details]
Broker3G_RilOn_TracesOn_master.txt
You need to log in before you can comment on or make changes to this bug.