From 8b3ff56d28abde42bb438eede67651042ca9a805 Mon Sep 17 00:00:00 2001 From: Lorenzo Colitti Date: Mon, 11 Apr 2016 17:08:10 +0900 Subject: [PATCH] If we want to release the lock, release instead of acquiring it. The intent of this code seems pretty clearly to release the lock, not acquire it. This is bad since it's a refcounted wakelock. Change-Id: I0f7590ab697133bd82db8a1a515453316c6a80dc --- .../core/java/com/android/server/NativeDaemonConnector.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/services/core/java/com/android/server/NativeDaemonConnector.java b/services/core/java/com/android/server/NativeDaemonConnector.java index 7db9be20e9..6009984842 100644 --- a/services/core/java/com/android/server/NativeDaemonConnector.java +++ b/services/core/java/com/android/server/NativeDaemonConnector.java @@ -243,7 +243,7 @@ final class NativeDaemonConnector implements Runnable, Handler.Callback, Watchdo log("Problem parsing message " + e); } finally { if (releaseWl) { - mWakeLock.acquire(); + mWakeLock.release(); } }