tree 28fe1f14a63d898dbac17b7b1ce7419009c734c8
parent 509678ce9efa2824092d3c2ddca04b8f88f31f61
author Todd Poynor <toddpoynor@google.com> 1498512878 -0700
committer android-build-team Robot <android-build-team-robot@google.com> 1498602993 +0000

healthd: notify listeners using local copy of list, drop lock

Binder currently may service an incoming oneway transaction whenever
an outbound oneway call is made (if there is already a pending
incoming oneway call waiting).  The unexpected nested method call
blocks forever on a recursive mutex acquire because healthd is single-
threaded.  The binder behavior is considered a bug and may change in
the future.  For now, work around this in healthd.

Make a local copy of the listeners list, then drop the lock and
perform the outbound calls on the local copy of the list.

Bug: 38201220
Test: Marlin with modified client calling scheduleUpdate() repeatedly
Change-Id: If35c2847556245921e2aff808ff747bb60356811
(cherry picked from commit fd68370074fab549f569d8b9a5bf3d7c535c47bc)