After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 723783 - Wrong Device.ActiveConnection value emitted when device starts activating
Wrong Device.ActiveConnection value emitted when device starts activating
Status: RESOLVED FIXED
Product: NetworkManager
Classification: Platform
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: NetworkManager maintainer(s)
NetworkManager maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2014-02-06 17:00 UTC by Dan Williams
Modified: 2014-03-21 00:25 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
0001-core-export-ActiveConnection-before-handing-it-to-th.patch (4.12 KB, patch)
2014-03-20 16:43 UTC, Dan Williams
accepted-commit_now Details | Review

Description Dan Williams 2014-02-06 17:00:40 UTC
signal sender=:1.10 -> dest=(null destination) serial=34390 path=/org/freedesktop/NetworkManager/Devices/2; interface=org.freedesktop.NetworkManager.Device.Wireless; member=PropertiesChanged
   array [
      dict entry(
         string "ActiveConnection"
         variant             object path "/"
      )
      dict entry(
         string "State"
         variant             uint32 40
      )
      dict entry(
         string "StateReason"
         variant             struct {
               uint32 40
               uint32 0
            }
      )
   ]
Comment 1 Dan Williams 2014-03-20 16:17:47 UTC
This issue was found in git master, but I tested 0.9.8 and confirmed it does not exist there.  In 0.9.8 the object is exported immediately after it's created, long before it gets assigned to the device, so 0.9.8 should always have a valid ActiveConnection path when the state changes to PREPARE.
Comment 2 Dan Williams 2014-03-20 16:43:27 UTC
Created attachment 272502 [details] [review]
0001-core-export-ActiveConnection-before-handing-it-to-th.patch
Comment 3 Dan Winship 2014-03-20 17:11:15 UTC
Comment on attachment 272502 [details] [review]
0001-core-export-ActiveConnection-before-handing-it-to-th.patch

makes sense
Comment 4 Thomas Haller 2014-03-20 17:50:42 UTC
looks good to me
Comment 5 Dan Williams 2014-03-21 00:25:59 UTC
Pushed to git master.