commit | ae107afe8e95b42543c05f67759a4ee32cfc6fb7 | [log] [tgz] |
---|---|---|
author | Lorenzo Colitti <lorenzo@google.com> | Mon Apr 11 17:08:10 2016 +0900 |
committer | Lorenzo Colitti <lorenzo@google.com> | Mon Apr 11 23:19:06 2016 +0900 |
tree | b55136a0eea0faeaf084b66d34bc5298253afc1e | |
parent | 39d2bb56b6a938ae053d96d385a68b858678d6ad [diff] |
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
diff --git a/services/core/java/com/android/server/NativeDaemonConnector.java b/services/core/java/com/android/server/NativeDaemonConnector.java index 7db9be2..6009984 100644 --- a/services/core/java/com/android/server/NativeDaemonConnector.java +++ b/services/core/java/com/android/server/NativeDaemonConnector.java
@@ -243,7 +243,7 @@ log("Problem parsing message " + e); } finally { if (releaseWl) { - mWakeLock.acquire(); + mWakeLock.release(); } }