From 09a177ec51d7c0764ac86d9a38e9d2e7d0c47262 Mon Sep 17 00:00:00 2001 From: Christian Muehlhaeuser <muesli@gmail.com> Date: Thu, 17 Jan 2013 07:47:17 +0100 Subject: [PATCH] * Don't crash in StreamConnection without m_iodev. --- src/libtomahawk/network/StreamConnection.cpp | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/libtomahawk/network/StreamConnection.cpp b/src/libtomahawk/network/StreamConnection.cpp index 470f2cd24..5ac9db28d 100644 --- a/src/libtomahawk/network/StreamConnection.cpp +++ b/src/libtomahawk/network/StreamConnection.cpp @@ -215,7 +215,7 @@ StreamConnection::handleMsg( msg_ptr msg ) else if ( msg->payload().startsWith( "doneblock" ) ) { int block = QString( msg->payload() ).mid( 9 ).toInt(); - ((BufferIODevice*)m_iodev.data())->seeked( block ); + ( (BufferIODevice*)m_iodev.data() )->seeked( block ); m_curBlock = block; qDebug() << "Next block is now:" << block; @@ -223,18 +223,21 @@ StreamConnection::handleMsg( msg_ptr msg ) else if ( msg->payload().startsWith( "data" ) ) { m_badded += msg->payload().length() - 4; - ((BufferIODevice*)m_iodev.data())->addData( m_curBlock++, msg->payload().mid( 4 ) ); + ( (BufferIODevice*)m_iodev.data() )->addData( m_curBlock++, msg->payload().mid( 4 ) ); } //qDebug() << Q_FUNC_INFO << "flags" << (int) msg->flags() // << "payload len" << msg->payload().length() // << "written to device so far: " << m_badded; - if ( ((BufferIODevice*)m_iodev.data())->nextEmptyBlock() < 0 ) + if ( !m_iodev || ( (BufferIODevice*)m_iodev.data() )->nextEmptyBlock() < 0 ) { m_allok = true; + // tell our iodev there is no more data to read, no args meaning a success: - ((BufferIODevice*)m_iodev.data())->inputComplete(); + if ( m_iodev ) + ( (BufferIODevice*)m_iodev.data() )->inputComplete(); + shutdown(); } }