From b6aab53d10d1db4808fd7cdaf248a91344a2d36a Mon Sep 17 00:00:00 2001
From: Giovanni Campagna <gcampagna@src.gnome.org>
Date: Thu, 29 Mar 2012 18:51:03 +0200
Subject: [PATCH] NetworkMenu: fix crash when NetworkManger restarts

nm_active_connection_get_devices() has a (questionable) special case
for the no devices case (which happens if the DBus object is
destroyed because NM went down): it returns null instead of an empty
array. Handle that instead of crashing.

https://bugzilla.gnome.org/show_bug.cgi?id=673043
---
 js/ui/status/network.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/js/ui/status/network.js b/js/ui/status/network.js
index 5c6babee8..f7d2258cf 100644
--- a/js/ui/status/network.js
+++ b/js/ui/status/network.js
@@ -1844,7 +1844,7 @@ const NMApplet = new Lang.Class({
                 if (a._type != NetworkManager.SETTING_VPN_SETTING_NAME) {
                     // find a good device to be considered primary
                     a._primaryDevice = null;
-                    let devices = a.get_devices();
+                    let devices = a.get_devices() || [];
                     for (let j = 0; j < devices.length; j++) {
                         let d = devices[j];
                         if (d._delegate) {