Correct last frame reflection issue (#1)

* Correct last frame reflection issue

There's a bug in this version that reflects the last frame to the sender, this is the change from upstream that corrects the issue.

* Update comment

Adding the comment from upstream.
pull/3/head
iamsi 5 years ago committed by GitHub
parent 7951de404b
commit 830edf8bc2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -183,8 +183,12 @@ void CProtocol::OnDvLastFramePacketIn(std::unique_ptr<CDvLastFramePacket> &Frame
stream->Push(std::move(Frame));
stream->Unlock();
// and close the stream
g_Reflector.CloseStream(stream);
// Don't close yet, this stops the last packet relfection bug that was fixed in upstream by the same change.
// Don't close the stream yet but rely on CheckStreamsTimeout
// mechanism, so the stream will be closed after the queues have
// been sinked out. This avoid last packets to be send back
// to transmitting client (master)
}
// else
// {

Loading…
Cancel
Save

Powered by TurnKey Linux.