commit 282c038240bba5fff2272f97eaf6f5e2443bfffc
author Tetsutoki Shiozawa <tetsutoki.shiozawa@sony.com> Mon Feb 26 07:43:18 2018 +0000
committer android-build-merger <android-build-merger@google.com> Mon Feb 26 07:43:18 2018 +0000
tree 7e5ead57f6d67191791ba25704bd927e8de52d15
parent 472d796ab2bb094fd587237292af718168fa787c
parent 2fd5af103609594f021e2d7d068df79c43d63d5a

Merge "Definitely abort reconnecting to native daemon during shutdown"
am: 2fd5af1036

Change-Id: I42d34accd9b56d1090a76632751447653cc8dd61

services/core/java/com/android/server/NativeDaemonConnector.java

diff --git a/services/core/java/com/android/server/NativeDaemonConnector.java b/services/core/java/com/android/server/NativeDaemonConnector.java
index b5a8332..ad02aad 100644
--- a/services/core/java/com/android/server/NativeDaemonConnector.java
+++ b/services/core/java/com/android/server/NativeDaemonConnector.java
@@ -134,21 +134,23 @@
         mCallbackHandler = new Handler(mLooper, this);
 
         while (true) {
+            if (isShuttingDown()) break;
             try {
                 listenToSocket();
             } catch (Exception e) {
                 loge("Error in NativeDaemonConnector: " + e);
-                String shutdownAct = SystemProperties.get(
-                        ShutdownThread.SHUTDOWN_ACTION_PROPERTY, "");
-                if (shutdownAct != null && shutdownAct.length() > 0) {
-                    // The device is in middle of shutdown.
-                    break;
-                }
+                if (isShuttingDown()) break;
                 SystemClock.sleep(5000);
             }
         }
     }
 
+    private static boolean isShuttingDown() {
+        String shutdownAct = SystemProperties.get(
+            ShutdownThread.SHUTDOWN_ACTION_PROPERTY, "");
+        return shutdownAct != null && shutdownAct.length() > 0;
+    }
+
     @Override
     public boolean handleMessage(Message msg) {
         final String event = (String) msg.obj;