diff options
| author | Jiří Klimeš <jklimes@redhat.com> | 2015-08-25 13:32:53 +0200 |
|---|---|---|
| committer | Jiří Klimeš <jklimes@redhat.com> | 2015-08-25 14:24:30 +0200 |
| commit | 6bb0d61183d209b1d7cee3d0504187c4a65ac931 (patch) | |
| tree | aa2031fee465e35da557882d4cf2d1d9ce6774a8 | |
| parent | 0bc335cfbee754b472686cbad96cb6ad6d9c6a0a (diff) | |
| download | NetworkManager-jk/configure-and-quit-fix.tar.gz | |
manager: don't set "startup complete" until devices have been addedjk/configure-and-quit-fix
check_if_startup_complete() could be invoked by nm_settings_start() before
devices had chance to be added, which results in premature quit.
Postpone the processing until we add all devices and they are processed.
(gdb) bt
#0 0x00005555556401f3 in check_if_startup_complete (self=0x5555559f91d0)
at nm-manager.c:719
#1 0x00007ffff4d69de8 in g_closure_invoke () at /lib64/libgobject-2.0.so.0
#2 0x00007ffff4d7b70d in signal_emit_unlocked_R ()
at /lib64/libgobject-2.0.so.0
#3 0x00007ffff4d83471 in g_signal_emit_valist () at /lib64/libgobject-2.0.so.0
#4 0x00007ffff4d8372f in g_signal_emit () at /lib64/libgobject-2.0.so.0
#5 0x00007ffff4d6e4b5 in g_object_dispatch_properties_changed ()
at /lib64/libgobject-2.0.so.0
#6 0x00007ffff4d709d9 in g_object_notify () at /lib64/libgobject-2.0.so.0
#7 0x00005555556e232c in check_startup_complete (self=self@entry=0x555555a0e130) at settings/nm-settings.c:204
#8 0x00005555556e5203 in nm_settings_start (self=0x555555a0e130, error=error@entry=0x7fffffffe658) at settings/nm-settings.c:2122
#9 0x0000555555646d06 in nm_manager_start (self=0x5555559f91d0, error=0x7fffffffe658) at nm-manager.c:4153
#10 0x00005555555add43 in main (argc=1, argv=0x7fffffffe7c8) at main.c:428
(gdb)
Fixes:Beaker:NetworkManager_Test37_run_once_new_connection
| -rw-r--r-- | src/nm-manager.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/nm-manager.c b/src/nm-manager.c index 5744a8cdc9..71b9d7074f 100644 --- a/src/nm-manager.c +++ b/src/nm-manager.c @@ -138,6 +138,7 @@ typedef struct { guint timestamp_update_id; gboolean startup; + gboolean devices_inited; } NMManagerPrivate; #define NM_MANAGER_GET_PRIVATE(o) (G_TYPE_INSTANCE_GET_PRIVATE ((o), NM_TYPE_MANAGER, NMManagerPrivate)) @@ -687,6 +688,9 @@ check_if_startup_complete (NMManager *self) if (!priv->startup) return; + if (!priv->devices_inited) + return; + if (!nm_settings_get_startup_complete (priv->settings)) { nm_log_dbg (LOGD_CORE, "check_if_startup_complete returns FALSE because of NMSettings"); return; @@ -4201,6 +4205,8 @@ nm_manager_start (NMManager *self, GError **error) */ system_create_virtual_devices (self); + priv->devices_inited = TRUE; + check_if_startup_complete (self); return TRUE; |