3987b6d - platform/external/webrtc

commit	3987b6de506a7e72a5bdfdf8c8ad9964705c5a28	[log] [tgz]
author	jiayl@webrtc.org <jiayl@webrtc.org>	Wed Sep 24 17:14:05 2014 +0000
committer	jiayl@webrtc.org <jiayl@webrtc.org>	Wed Sep 24 17:14:05 2014 +0000
tree	d7b7389b4bc579aaed4a7bdca6f11ad62d05217e
parent	a0ce9fa2a65a693ca9a6ee4920fb41d5cdd92e3b [diff]

Fix a problem in Thread::Send.
Previously if thread A->Send is called on thread B, B->ReceiveSends will be called, which enables an arbitrary thread to invoke calls on B while B is wait for A->Send to return. This caused mutliple problems like issue 3559, 3579.
The fix is to limit B->ReceiveSends to only process requests from A.
Also disallow the worker thread invoking other threads.

BUG=3559
R=juberti@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/15089004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@7290 4adac7df-926f-26a2-2b94-8c16560cd09d

talk/app/webrtc/peerconnection_unittest.cc[diff]
talk/app/webrtc/peerconnectionfactory.cc[diff]
talk/app/webrtc/test/peerconnectiontestwrapper.cc[diff]
talk/app/webrtc/test/peerconnectiontestwrapper.h[diff]
talk/session/media/channelmanager.cc[diff]
webrtc/base/thread.cc[diff]
webrtc/base/thread.h[diff]
webrtc/base/thread_unittest.cc[diff]

8 files changed

tree: d7b7389b4bc579aaed4a7bdca6f11ad62d05217e