mirror/tomahawk
1
0
Fork 0
mirror of https://github.com/tomahawk-player/tomahawk.git synced 2025-03-23 17:29:42 +01:00
Code Issues Releases Wiki Activity

* Use a QWeakPointer to guard Connection pointers.

Browse Source
This commit is contained in:
Christian Muehlhaeuser 2011-09-13 10:47:10 +02:00
parent 48e3eb877d
commit 45b57b715f
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/libtomahawk/network/servent.cpp
View File

@ -241,7 +241,7 @@ Servent::setExternalAddress( QHostAddress ha, unsigned int port )
void
Servent::registerOffer( const QString& key, Connection* conn )
{
m_offers[key] = QPointer<Connection>(conn);
m_offers[key] = QWeakPointer<Connection>(conn);
}
@ -629,7 +629,7 @@ Servent::claimOffer( ControlConnection* cc, const QString &nodeid, const QString
if( m_offers.contains( key ) )
{
QPointer<Connection> conn = m_offers.value( key );
QWeakPointer<Connection> conn = m_offers.value( key );
if( conn.isNull() )
{
// This can happen if it's a streamconnection, but the audioengine has
@ -642,21 +642,21 @@ Servent::claimOffer( ControlConnection* cc, const QString &nodeid, const QString
if( !nodeid.isEmpty() )
{
// If there isn't a nodeid it's not the first connection and will already have been stopped
if( !checkACL( conn, nodeid, true ) )
if( !checkACL( conn.data(), nodeid, true ) )
{
tLog() << "Connection not allowed due to ACL";
return NULL;
}
}
if( conn->onceOnly() )
if( conn.data()->onceOnly() )
{
m_offers.remove( key );
return conn.data();
}
else
{
return conn->clone();
return conn.data()->clone();
}
}
else if ( noauth )

2
src/libtomahawk/network/servent.h
View File

@ -158,7 +158,7 @@ private:
QJson::Parser parser;
QList< ControlConnection* > m_controlconnections; // canonical list of authed peers
QMap< QString, QPointer<Connection> > m_offers;
QMap< QString, QWeakPointer<Connection> > m_offers;
int m_port, m_externalPort;
QHostAddress m_externalAddress;
QString m_externalHostname;