diff options
| author | Pasi Petäjäjärvi <pasi.petajajarvi@qt.io> | 2022-05-12 15:45:04 +0300 |
|---|---|---|
| committer | Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> | 2022-06-14 11:49:09 +0000 |
| commit | 32b905a322ffdd34f15bcd973985bef388a6d12a (patch) | |
| tree | 9c2feb89ef795b7474d3a8eefa49515e10c3c72a /src/networksettings/connman/qnetworksettingsmanager_p.cpp | |
| parent | 718db8993e66d87862ecd441a8b89c7d62adb215 (diff) | |
Fix crash when unplugging/plugging ethernet cable
We do not remove the ethernet service (network) when
cable is disconnected. Instead it's "hold in place"
via placeholder but that is not checked and plugging
cable back will cause crash.
Fixes: QTBUG-103327
Change-Id: If14a4f28217ba5883dbc23ba42a1e91b004b7d58
Reviewed-by: Samuli Piippo <samuli.piippo@qt.io>
(cherry picked from commit 26b002e1388d197772b56d471a6dd6ecc4ab2509)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
Diffstat (limited to 'src/networksettings/connman/qnetworksettingsmanager_p.cpp')
| -rw-r--r-- | src/networksettings/connman/qnetworksettingsmanager_p.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/networksettings/connman/qnetworksettingsmanager_p.cpp b/src/networksettings/connman/qnetworksettingsmanager_p.cpp index 836adc7..770cb44 100644 --- a/src/networksettings/connman/qnetworksettingsmanager_p.cpp +++ b/src/networksettings/connman/qnetworksettingsmanager_p.cpp @@ -260,9 +260,11 @@ void QNetworkSettingsManagerPrivate::onServicesChanged(ConnmanMapStructList chan emit q->servicesChanged(); if (auto service = m_unnamedServices.take(path)) - service->deleteLater(); + if (!service->placeholderState()) + service->deleteLater(); if (auto service = m_unknownServices.take(path)) - service->deleteLater(); + if (!service->placeholderState()) + service->deleteLater(); m_unnamedServicesForSsidConnection.remove(path); // do not delete here } |