Sim blocked dialogue broken on starting phone

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] ...

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

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.

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.

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.

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

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...