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 722785 - NMClient should emit 'device-removed' for all devices on dispose
NMClient should emit 'device-removed' for all devices on dispose
Status: RESOLVED OBSOLETE
Product: NetworkManager
Classification: Platform
Component: API
unspecified
Other Linux
: Normal normal
: ---
Assigned To: NetworkManager maintainer(s)
NetworkManager maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2014-01-22 18:00 UTC by Zeeshan Ali
Modified: 2020-11-12 14:25 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Zeeshan Ali 2014-01-22 18:00:38 UTC
From IRC:

<zeenix> dcbw: unrefing NMClient calls device-removed for all devices?
<dcbw> zeenix: I'll assume you mean the last ref?
<zeenix> dcbw: yeah and i just checked the code to find out the answer: yes
<zeenix> dcbw: oh i misread the code, it seems device-removed signal is not called :(
<zeenix> dcbw: i think it should
<zeenix> in my device-removed handler, i reset signals handler ids and since this signal is not called on client discpose, i have to do the same on discposing client
<dcbw> zeenix: free_devices(client, FALSE); should be ensuring the signal doesn't got out I think... though if you have references on the devices, they won't go away?
<dcbw> zeenix: so is the issue that when you're cleaning stuff up, it's more code then to go through and clean up every device instead of listening to the signal (which you're already listening to)?
<dcbw> hmm, looks liek a general decision in libnm-glib not to call 'removed' signals unless the object was actually removed by the daemon
<dcbw> zeenix: would you mind filing a bugzilla.gnome.org bug saying this?  we could fix it when we do a libnm-glib2 to kill dbus-glib or perhaps with a "cleanup" signal for child objects
Comment 1 André Klapper 2020-11-12 14:25:21 UTC
bugzilla.gnome.org is being shut down in favor of a GitLab instance. 
We are closing all old bug reports and feature requests in GNOME Bugzilla which have not seen updates for a long time.

If you still use NetworkManager and if you still see this bug / want this feature in a recent and supported version of NetworkManager, then please feel free to report it at https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/issues/

Thank you for creating this report and we are sorry it could not be implemented (workforce and time is unfortunately limited).