[SIM PIN]+[krillin] cannot disable SIM locking via system-settings if SIM locked

Bug #1375945 reported by Bill Filler
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ubuntu UX
Invalid
High
Matthew Paul Thomas
ofono (Ubuntu)
Fix Released
High
Alfonso Sanchez-Beato
ubuntu-system-settings (Ubuntu)
Invalid
Undecided
Unassigned

Bug Description

rtm build 74 on krillin

After setting up a sim lock, and rebooting my phone, I cannot unlock the sims from the system settings. I can unlock from the network menu and only after doing so, then I can unlock from system settings.

Steps to reproduce:
1) Enable sim lock for both sims (I'm testing dual-sim, guessing same problem happens with one sim) by going system settings->security and privacy->SIM PIN and enabling the toggle.
2) you will be prompted for the current password for the sim (if you've never set it before it does still exist and is set by the carrier. for at+t it is 1111 for others I've heard 0000 or 1234) and then set a new password of something different
3) reboot your phone
4) ensure that network indicator showing the sims as locked (see icon and menu items in indicator)
5) now try to unlock by going to system settings->security and privacy->SIM PIN and disabling the toggle.
6) you are prompted for the pin
7) enter the pin you setup in step 2

Expected result:
the sim should now be unlocked

Actual result:
the dialog says "invalid password, 3 attempts remaining". try another time and get the same message

8) Now, go to the network menu and press the sim unlock button
9) At the prompt, enter the sim pin
10) the sim is correctly unlocked, but this is only for the session until you reboot.
11) now if you go to system settings and perform step 5-7 and it works correctly and the toggle stays off and sim truly unlocked after reboot.

Tags: rtm14

Related branches

Bill Filler (bfiller)
Changed in ubuntu-system-settings (Ubuntu):
importance: Undecided → Critical
tags: added: rtm14
Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

The action in settings is really to disable locking. So the user must first unlock the sim from the menu, then can "dsisble locking" (not unlock) for that sim.

One solution would be for settings to do both actions which is what the user would expect since the PIN they enter is correct.
The current message that the PIN is invalid is certainly incorrect and leads the user to think they forgot their PIN.

Revision history for this message
Tony Espy (awe) wrote :

Same error running krillin + rtm #72.

Tried to toggle the toggle the SIM PIN control for SIM 1 which was locked. Was prompted to enter my PIN and when I did, I saw the same error dialog ( see attached screenshot ) as Bill.

Note, I had a second unlocked SIM in slot #2.

Changed in ubuntu-system-settings (Ubuntu):
status: New → Confirmed
Revision history for this message
Tony Espy (awe) wrote :

So a bit of digging shows that this looks to be device-specific.

On krillin, I'm unable to unlock the SIM unless I've previously entered the SIM PIN ( ie. the SIM has already been entered once ).

On mako, the same operation succeeds without having previously entered the SIM PIN.

I've added an ofono task and assigned to Alfonso for a second opinion.

Changed in ofono (Ubuntu):
assignee: nobody → Alfonso Sanchez-Beato (alfonsosanchezbeato)
Changed in ubuntu-system-settings (Ubuntu):
assignee: nobody → Ken VanDine (ken-vandine)
importance: Critical → Undecided
Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

First, as Pat commented there has been some confusion in the bug between actions:

Enter PIN -> Enter PIN after modem reboot to be able to use the phone
Unlock PIN -> Deactivate PIN so no PIN needs to be entered after modem reboot

For both actions you need to use the current PIN. Maybe some rewording of these actions in the GUI can remove the confusion for the user (Enter PIN vs Activate/deactivate PIN?)

Second, Tony is right and the behaviour for the "Unlock PIN" action depends on the device if we still have not entered the PIN ("Enter PIN") after a reboot. Using ofono scripts:

/usr/share/ofono/scripts/unlock-pin /ril_0 pin XXXX

works for mako, but for krillin we have to do

/usr/share/ofono/scripts/enter-pin /ril_0 pin XXXX
/usr/share/ofono/scripts/unlock-pin /ril_0 pin XXXX

TBH krillin behaviour is what I would have expected to happen usually, and of course doing the two steps for mako works too (mako's modem FW is probably doing itself the two steps if needed). So I would assume krillin behaviour as the default.

From a user perspective, my opinion is that displaying an error when the user tries to unlock the PIN saying "you need to enter the PIN first" helps the user understand the difference between entering PIN and PIN activation. Note also that this problem does not happen for non-Ubuntu phones, as afaik in all of them you have to enter the PIN on start to use the phone if it is activated.

Revision history for this message
Tony Espy (awe) wrote :

@Alfonso

Thanks for the detailed response.

With respect to other phones, the only other phone OS where I've used a locked SIM is Android. I'm pretty sure you it was possible in the past to skip the Unlock SIM screen and continue, but that doesn't appear to be the case with 4.4.4 on a Nexus 5.

AFAIK, the original Greeter design also behaved the same way, however I'm not aware of plans to change this before RTM, which means the user could end up in a state where they have a locked SIM in the device, and yet are still using it.

That said, I disagree with the comment about displaying an error that says "you need to enter a PIN first". "Enter PIN" and "Unlock PIN" both require the same PIN, so asking the user to enter the same string of digits twice doesn't make any sense.

As mako runs vanilla Android, I would also take it as the reference and claim that this should be quirked in the mtkmodem driver, or we modify the settings code to always perform both operations when unlocking the SIM.

I will invite Ken to our weekly network/telephony hangout and add this bug to the agenda.

Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

Our implementation will again prompt the user for the SIM Pin on a boot, but it will be possible to continue without entering it.
There is also a discussion ongoing to make it more obvious to the user that the SIM is locked (i.e. fix the indicator)
I would prefer a fix at the lower level rather than in the app.

Tony Espy (awe)
Changed in ofono (Ubuntu):
status: New → Confirmed
Tony Espy (awe)
summary: - cannot unlock SIM using system-settings
+ [krillin] cannot disable SIM locking via system-settings if SIM locked
Revision history for this message
Tony Espy (awe) wrote : Re: [krillin] cannot disable SIM locking via system-settings if SIM locked

@Pat

I agree that this should be quirked in the ofono mtkmodem implementation. Let's give Alfonso a chance to revisit and we should have an answer tomorrow on which approach we take.

Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

Looks like ofono core allowed to do the unlock even if the PIN has not been entered, so probably implementing the quirk in ofono is fine.

Tony Espy (awe)
Changed in ofono (Ubuntu):
importance: Undecided → High
Changed in ubuntu-system-settings (Ubuntu):
status: Confirmed → Invalid
assignee: Ken VanDine (ken-vandine) → nobody
Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

This branch homogenizes the behaviour for krillin/mako:

https://github.com/rilmodem/ofono/pull/127

Changed in ofono (Ubuntu):
status: Confirmed → In Progress
Tony Espy (awe)
Changed in ofono (Ubuntu):
status: In Progress → Fix Committed
summary: - [krillin] cannot disable SIM locking via system-settings if SIM locked
+ [SIM PIN]+[krillin] cannot disable SIM locking via system-settings if
+ SIM locked
Changed in ubuntu-ux:
assignee: nobody → Matthew Paul Thomas (mpt)
importance: Undecided → High
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ofono - 1.12.bzr6880+14.10.20141010-0ubuntu1

---------------
ofono (1.12.bzr6880+14.10.20141010-0ubuntu1) utopic; urgency=medium

  [ Alfonso Sanchez-Beato ]
  * drivers/rilmodem/sim.c: enter PIN before unlocking (LP: #1375945)
    Some modems require the PIN to be unlocked with a separate
    command before the PIN lock can be disabled.

  * src/sim.c: fix sim atom teardown crash (LP: #1374418)
 -- Ubuntu daily release <email address hidden> Fri, 10 Oct 2014 08:53:32 +0000

Changed in ofono (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Matthew Paul Thomas (mpt) wrote :

Since this is Fixed in one place and Invalid in the other, this doesn't seem to have required any design in the first place. Please reopen with more details if I'm mistaken.

Changed in ubuntu-ux:
status: New → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.