diff --git a/xbmc/dialogs/GUIDialogMediaSource.cpp b/xbmc/dialogs/GUIDialogMediaSource.cpp index e0e015972c3e4..1e3be6ffcec70 100644 --- a/xbmc/dialogs/GUIDialogMediaSource.cpp +++ b/xbmc/dialogs/GUIDialogMediaSource.cpp @@ -165,8 +165,7 @@ bool CGUIDialogMediaSource::ShowAndAddMediaSource(const CStdString &type) bool CGUIDialogMediaSource::ShowAndEditMediaSource(const CStdString &type, const CStdString&share) { - VECSOURCES* pShares=NULL; - + VECSOURCES* pShares = g_settings.GetSourcesFromType(type); if (pShares) { for (unsigned int i=0;isize();++i) @@ -175,7 +174,6 @@ bool CGUIDialogMediaSource::ShowAndEditMediaSource(const CStdString &type, const return ShowAndEditMediaSource(type,(*pShares)[i]); } } - return false; } diff --git a/xbmc/interfaces/legacy/CallbackHandler.cpp b/xbmc/interfaces/legacy/CallbackHandler.cpp index 38b438850b54f..0cedb85c7c2e5 100644 --- a/xbmc/interfaces/legacy/CallbackHandler.cpp +++ b/xbmc/interfaces/legacy/CallbackHandler.cpp @@ -145,11 +145,11 @@ namespace XBMCAddon #ifdef ENABLE_TRACE_API CLog::Log(LOGDEBUG,"%sNEWADDON removing callback 0x%lx for PyThreadState 0x%lx from queue", _tg.getSpaces(),(long)(p->cb.get()) ,(long)userData); #endif - g_callQueue.erase(iter); + iter = g_callQueue.erase(iter); } else iter++; - } + } } } diff --git a/xbmc/music/tags/TagLoaderTagLib.cpp b/xbmc/music/tags/TagLoaderTagLib.cpp index 48dcd4d90b4ff..56be51b890bcf 100644 --- a/xbmc/music/tags/TagLoaderTagLib.cpp +++ b/xbmc/music/tags/TagLoaderTagLib.cpp @@ -152,6 +152,7 @@ bool CTagLoaderTagLib::Load(const string& strFileName, CMusicInfoTag& tag, Embed if (!file || !file->isValid()) { delete file; + oggFlacFile = NULL; file = oggVorbisFile = new Ogg::Vorbis::File(stream); } } diff --git a/xbmc/network/WebServer.cpp b/xbmc/network/WebServer.cpp index ab0dd0c17007b..4a0ea86819bf7 100644 --- a/xbmc/network/WebServer.cpp +++ b/xbmc/network/WebServer.cpp @@ -395,7 +395,11 @@ int CWebServer::CreateFileDownloadResponse(struct MHD_Connection *connection, co &CWebServer::ContentReaderCallback, file, &CWebServer::ContentReaderFreeCallback); if (response == NULL) + { + file->Close(); + delete file; return MHD_NO; + } } else { diff --git a/xbmc/network/linux/ZeroconfAvahi.cpp b/xbmc/network/linux/ZeroconfAvahi.cpp index a849b9ebc1132..8214bb4021349 100644 --- a/xbmc/network/linux/ZeroconfAvahi.cpp +++ b/xbmc/network/linux/ZeroconfAvahi.cpp @@ -387,7 +387,7 @@ void CZeroconfAvahi::addService(tServiceMap::mapped_type fp_service_info, AvahiC { if ((ret = avahi_entry_group_add_service_strlst(fp_service_info->mp_group, AVAHI_IF_UNSPEC, AVAHI_PROTO_UNSPEC, AvahiPublishFlags(0), fp_service_info->m_name.c_str(), - fp_service_info->m_type.c_str(), NULL, NULL, fp_service_info->m_port, fp_service_info->mp_txt) < 0)) + fp_service_info->m_type.c_str(), NULL, NULL, fp_service_info->m_port, fp_service_info->mp_txt)) < 0) { if (ret == AVAHI_ERR_COLLISION) { diff --git a/xbmc/utils/BitstreamConverter.cpp b/xbmc/utils/BitstreamConverter.cpp index a169a322c5238..0bd407fb05de9 100644 --- a/xbmc/utils/BitstreamConverter.cpp +++ b/xbmc/utils/BitstreamConverter.cpp @@ -535,7 +535,10 @@ void CBitstreamConverter::Close(void) m_sps_pps_context.sps_pps_data = NULL; } if(m_convertBuffer) + { free(m_convertBuffer); + m_convertBuffer = NULL; + } m_convertSize = 0; }