jbagg/QtZeroConf
1
0
Fork 1
mirror of https://github.com/jbagg/QtZeroConf.git synced 2025-09-26 04:50:56 +02:00
Code Issues Packages Projects Releases Wiki Activity

Bonjour - destroy resolvers by resolvers list, not service list

Browse Source 
If stopBrowser is called after a resolve starts, but before the resolve
finished, the service is never added to the services list.  This leaves the
resolver active and it fires when resolving completes after stopBrowser()
has destroyed a bunch of objects.  Fix is to destroy resolves according to
resolvers list and destroy services according to services list.
This commit is contained in:
Jonathan Bagg
2025-09-15 16:01:36 -04:00
parent e7b95be255
commit 4a2e197517
1 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
bonjour.cpp
View File

@@ -239,12 +239,11 @@ void QZeroConfPrivate::cleanUp(DNSServiceRef toClean)
else if (toClean == browser) { else if (toClean == browser) {
browser = nullptr; browser = nullptr;
browserNotifier.clear(); browserNotifier.clear();
QMap<QString, QZeroConfService >::iterator i; for (auto resolver : resolvers)
for (i = pub->services.begin(); i != pub->services.end(); i++) { resolver->cleanUp();
QString key = (*i)->name() + QString::number((*i)->interfaceIndex()); resolvers.clear();
resolvers[key]->cleanUp(); for (auto service : pub->services)
emit pub->serviceRemoved(*i); emit pub->serviceRemoved(service);
}
pub->services.clear(); pub->services.clear();
} }
else if (toClean == dnssRef) { else if (toClean == dnssRef) {