Sim blocked dialogue broken on starting phone

[Dave Morley]

STEPS:
1. Start the phone with a sim you are prepared to scrap
2. Input the incorrect sim pin 3 times
3. Input the incorrect Puk code and new sim pin 10 times

EXPECTED:
I expect to see a page similar to the pin unlock page, only announcing that the sim is blocked.

ACTUAL:
See screenshots.

I get a light grey page with the correct text and then I can't do anything with the phone. I can't swipe the page away as there is no unity8, I can't swipe down the indcators or in the launcher so I'm stuck, I can't make an emergency call which makes this a critical ship blocker.

[Alfonso Sanchez-Beato (alfonsosanchezbeato)]

In fact I would not expect neither of the screenshots in comments #2 or #3, but something saying that the SIM is unrecoverable AND a way to dismiss that warning and enter the phone.

The SIM is unrecoverable when following ofono properties have values:

org.ofono.SimManager.PinRequired = puk
org.ofono.SimManager.Retries = ... [puk = 0] ...

[Dave Morley (davmor2)]

Set to critical as you can't access the phone or make emergency calls which I think is a legal requirement.

[Alfonso Sanchez-Beato (alfonsosanchezbeato)]

The bug (which leds to screenshot in comment #3) has been produced with a modified ofono that was fixing bug #1436820. This modified ofono shows the number of retries on boot and when the SIM gets toasted after introducing 10 times a wrong PUK (both things were not happening before).

Package attached to make testing easier for indicator-network or other packages affected.

[Pete Woods (pete-woods)]

It turns out the underlying cause of this was that the indicator was going into an infinite loop and then terminating. The notification service then doesn't track its clients well (at all) and doesn't clear up the fullscreen notification when the indicator dies.

So far we have a fix for the infinite loop in the indicator. But next we need to make some enhancements to the behaviour of the notification service.

[Pete Woods (pete-woods)]

Indeed, the notification service tracking is wobbly enough that when we fixed the indicator's behaviour to clean up after itself on termination (by closing the notification), subsequent runs could no-longer show notifications.

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[Launchpad Janitor (janitor)]

This bug was fixed in the package indicator-network - 0.5.1+15.10.20150519-0ubuntu1

---------------
indicator-network (0.5.1+15.10.20150519-0ubuntu1) wily; urgency=medium

  [ Antti Kaijanmäki ]
  * Fix .pc file and docs. (LP: #1456307)

indicator-network (0.5.1+15.04.20150513.1-0ubuntu1) vivid; urgency=medium

  [ Pete Woods ]
  * Add flight mode and wifi toggles to connectivity service interface
    removed: tests/integration/CMakeLists.txt
    tests/integration/connectivity-cpp/ tests/integration/connectivity-
    cpp/CMakeLists.txt tests/integration/connectivity-cpp/mocks/
    tests/integration/connectivity-cpp/mocks/urfkill.h
    tests/integration/connectivity-cpp/sig_term_catcher.h
    tests/integration/connectivity-cpp/test_data.h.in
    tests/integration/connectivity-cpp/test_flight_mode.cpp
    tests/integration/connectivity-cpp/test_networking_status.cpp
    tests/integration/connectivity-cpp/test_service.h
    tests/integration/connectivity-cpp/test_wifi_connect.cpp
    tests/integration/connectivity-cpp/test_wifi_link.cpp
    tests/integration/connectivity-cpp/test_wifi_link_dedup.cpp
    tests/integration/connectivity-cpp/test_wifi_link_enable_disable.cpp
    tests/integration/indicator/ tests/unit/indicator/CMakeLists.txt
    tests/unit/indicator/menuitems/CMakeLists.txt tests/unit/menumodel-
    cpp/CMakeLists.txt tests/unit/secret-agent/CMakeLists.txt
    tests/unit/test_variant.h added:
    data/org.freedesktop.DBus.Properties.xml src/connectivity-api/
    src/connectivity-api/CMakeLists.txt src/connectivity-
    api/connectivity-qml/ src/connectivity-api/connectivity-
    qml/CMakeLists.txt src/connectivity-api/connectivity-qt/
    src/connectivity-api/connectivity-qt/CMakeLists.txt
    src/connectivity-api/connectivity-qt/connectivityqt/
    src/connectivity-api/connectivity-qt/connectivityqt/connectivity.cpp
    src/connectivity-api/connectivity-qt/connectivityqt/connectivity.h
    src/connectivity-api/connectivity-qt/connectivityqt/internal/
    src/connectivity-api/connectivity-qt/connectivityqt/internal/dbus-
    property-cache.cpp src/connectivity-api/connectivity-
    qt/connectivityqt/internal/dbus-property-cache.h
    tests/integration/indicator-network-test-base.cpp
    tests/integration/indicator-network-test-base.h
    tests/integration/test-connectivity-api.cpp renamed:
    src/connectivity-cpp/ => src/connectivity-api/connectivity-cpp/
    src/qdbus-stubs/DBusTypes.h => src/qdbus-stubs/dbus-types.h
    tests/integration/indicator/CMakeLists.txt =>
    tests/integration/CMakeLists.txt
    tests/integration/indicator/TestIndicatorNetworkService.cpp =>
    tests/integration/test-indicator.cpp tests/unit/secret-
    agent/TestSecretAgent.cpp => tests/unit/secret-agent/test-secret-
    agent.cpp
  * Fixed some clang compiler warnings. Fixed a dodgey self-comparison
    in operator<(const Key &other). Added some const-correctness here
    and there. Removed an unused maxRetries variable from sim-unlock-
    dialog. Replaced #ifndef's with #pragma once. Rename "i-n-
    extractor.cpp" to "i-n-sniffer.cpp". renamed: src/sniffer/i-n-
    extractor.cpp => src/sniffer/i-n-sniffer.cpp
  * Improve grouped access point hand...