ANDROID: binder: don't queue async transactions to thread.
authorMartijn Coenen <maco@android.com>
Thu, 10 Aug 2017 11:56:16 +0000 (13:56 +0200)
committerMartijn Coenen <maco@android.com>
Fri, 11 Aug 2017 08:23:20 +0000 (10:23 +0200)
commit4501c044de528a95de90de6a53e0af8c3a1a7207
treeb0520c88a9893cdad4274aa5dc4d12d7e7f6a876
parent3bdbe4c16b7b75941b4e03be4f59c3aa78ed9137
ANDROID: binder: don't queue async transactions to thread.

This can cause issues with processes using the poll()
interface:

1) client sends two oneway transactions
2) the second one gets queued on async_todo
   (because the server didn't handle the first one
    yet)
3) server returns from poll(), picks up the
   first transaction and does transaction work
4) server is done with the transaction, sends
   BC_FREE_BUFFER, and the second transaction gets
   moved to thread->todo
5) libbinder's handlePolledCommands() only handles
   the commands in the current data buffer, so
   doesn't see the new transaction
6) the server continues running and issues a new
   outgoing transaction. Now, it suddenly finds
   the incoming oneway transaction on its thread
   todo, and returns that to userspace.
7) userspace does not expect this to happen; it
   may be holding a lock while making the outgoing
   transaction, and if handling the incoming
   trasnaction requires taking the same lock,
   userspace will deadlock.

By queueing the async transaction to the proc
workqueue, we make sure it's only picked up when
a thread is ready for proc work.

Bug: 38201220
Bug: 63075553
Bug: 63079216

Change-Id: I84268cc112f735d7e3173793873dfdb4b268468b
Signed-off-by: Martijn Coenen <maco@android.com>
drivers/android/binder.c