network-manager crash on boot on krillin/devel-proposed

[Jean-Baptiste Lallement]

current build number: 198
device name: krillin
channel: ubuntu-touch/devel-proposed/krillin.en

network-manager:
  Installed: 1.0.4-0ubuntu3

network-manager crashes and keeps re-spawning, making the device unusable.

[John McAleely (john.mcaleely)]

@awe, it seems I lack assign-fu. can you take a look at this, but not ahead of the ota-7 wifi related bugs?

[Francis Ginther (fginther)]

This is currently blocking all boottesting for package promotion into wily. For reference, see the packages marked with "Boottest result: Regression" in http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html

[Tony Espy (awe)]

Confirmed on krillin ( devel-proposed #206 ). The .crash file wasn't very useful, but I should be able to diagnose fairly quick.

[Tony Espy (awe)]

Here's the backtrace:

#0 0x000aa24c in platform_link_added (self=self@entry=0x20bf930, ifindex=<optimized out>, plink=plink@entry=0x20cbfc8) at nm-manager.c:1956
        ignore = 0
        factory = <optimized out>
        device = <optimized out>
        error = 0x0
        __func__ = "platform_link_added"
#1 0x000ac02a in platform_query_devices (self=0x20bf930) at nm-manager.c:2070
        links_array = 0x2089420
        links = <optimized out>
        i = 3
#2 nm_manager_start (self=self@entry=0x20bf930) at nm-manager.c:4293
        priv = <optimized out>
        __func__ = "nm_manager_start"
#3 0x0003c662 in main (argc=1, argv=0xbef66e14) at main.c:500
        wifi_enabled = <optimized out>
        net_enabled = <optimized out>

[Tony Espy (awe)]

The code in question is this:

/* Try registered device factories */
factory = nm_device_factory_manager_find_factory_for_link_type (plink->type);
if (factory) {
    gboolean ignore = FALSE;

    device = nm_device_factory_new_link (factory, plink, &ignore, &error);
    if (!device) {
        if (!ignore) {
            nm_log_warn (LOGD_HW, "%s: factory failed to create device: %s", plink->name, error->message);
             g_clear_error (&error);
        }
        return;
    }
}

The crash is happening when error->message is accessed in the call to nm_log_warn.

[Tony Espy (awe)]

No crash on mako ( devel-proposed #313 ).

Pretty sure this is being caused by a device factory failure when trying to create a NMDevice instance for 'ccmni0', which is modem device, managed by rild.

[Tony Espy (awe)]

NM 0.9.10.0 included an "?" ( if ) check in the log statement that's crashing. Apparently this got lost in the merge. Patch on the way...

[Tony Espy (awe)]

Also, testing on krillin ( devel-proposed #225 ) is super painful. Unity seems to slow to crawl while painting the Welcome screen. It take several minutes for the phone to become usable... and spends much of the time prior to this basically frozen on the lock screen.

[Tony Espy (awe)]

Fix should land in 1.0.4-0ubuntu4, which is currently in the unapproved queue. Hopefully should land soon.

[Launchpad Janitor (janitor)]

This bug was fixed in the package network-manager - 1.0.4-0ubuntu4

---------------
network-manager (1.0.4-0ubuntu4) wily; urgency=medium

  [ Mathieu Trudel-Lapierre ]
  * d/p/0001-wwan-add-support-for-using-oFono-as-a-modem-manager.patch:
    Pre-initialize reason for ofono state changes to NULL; this avoids getting
    to nm_device_set_state with an invalid reason string. (LP: #1499906)
  * d/p/0001-wwan-add-support-for-using-oFono-as-a-modem-manager.patch:
    cleanup ip_string_to_network_address() to work as its NMModemBroadband
    version, and start setting NM_MODEM_IP4_METHOD to _STATIC so IP, prefix,
    gateway and DNS get properly applied on an oFono modem as it comes up.
  * debian/NetworkManager.conf: re-enable the ofono plugin (fixes a regression).
  * debian/patches/add_ofono_settings_support.patch: don't link against
    libnm-glib as this would cause type duplication and crashes. (LP: #1499904)
  * debian/patches/lp1496434_warn_on_invalid_new_link.patch: don't outright
    crash by dereferencing a NULL pointer if a device can't get created by its
    NMDeviceFactory; we should warn though, as we want to know about these
    failures; but no need to make all of NM sad. (LP: #1496434)

  [ Tony Espy ]
  * d/p/0002-wifi-cull-the-scan-list-before-signalling-ScanDone-b.patch:
    Fix duplicate 'LastSeen' PropertiesChanged signals being generated
    after every scan. The 'Last-Seen' property is now only updated when
    a scan finishes, and schedule_scanlist_cull is no longer triggered
    new_bss_cb or updated_bss_cb. (LP: #1480877)
  * d/p/add_ofono_settings_support.patch: Fix memory leaks in plugin.
  * d/p/add_ofono_settings_support.patch: Remove unused plugin DBus code.
  * d/p/CVE-2015-1322.patch: Drop CVE which applied to removed DBus code
    in ofono settings plugin.

 -- Mathieu Trudel-Lapierre <email address hidden> Sat, 26 Sep 2015 00:02:02 -0400