Browse Source

Merge pull request #365 from stazio/fix_media_handler

Added a move to the MediaHandler and a null test
Paul-Louis Ageneau 4 years ago
parent
commit
6e70ef35ba
1 changed files with 3 additions and 2 deletions
  1. 3 2
      src/impl/track.cpp

+ 3 - 2
src/impl/track.cpp

@@ -182,10 +182,11 @@ bool Track::transportSend([[maybe_unused]] message_ptr message) {
 void Track::setMediaHandler(shared_ptr<MediaHandler> handler) {
 	{
 		std::unique_lock lock(mMutex);
-		mMediaHandler = handler;
+		mMediaHandler = std::move(handler);
 	}
 
-	handler->onOutgoing(std::bind(&Track::transportSend, this, std::placeholders::_1));
+	if (mMediaHandler)
+		mMediaHandler->onOutgoing(std::bind(&Track::transportSend, this, std::placeholders::_1));
 }
 
 shared_ptr<MediaHandler> Track::getMediaHandler() {