commit 7830ef3dd0ff3749d974c2dd85a8fa59dc47aeca
author Aravind Akella <aakella@google.com> Tue Oct 07 14:13:12 2014 -0700
committer Aravind Akella <aakella@google.com> Tue Oct 07 21:20:12 2014 +0000
tree e6ad68ea20f80b48138a310b50b9cea44dce3afa
parent 78639c22acfa5e56d87fd6fe8340d083536b29fc

Change ordering of memory allocation and calling Thread::run().

In some cases this is causing a crash as device.poll is called with
NULL.

Bug: 17896339
Change-Id: Id431599f2c661338c355c7081b6602f8449a9198

services/sensorservice/SensorService.cpp

diff --git a/services/sensorservice/SensorService.cpp b/services/sensorservice/SensorService.cpp
index 8831893..0f945f7 100644
--- a/services/sensorservice/SensorService.cpp
+++ b/services/sensorservice/SensorService.cpp
@@ -185,14 +185,14 @@
             }
 
             mWakeLockAcquired = false;
-            run("SensorService", PRIORITY_URGENT_DISPLAY);
             mLooper = new Looper(false);
-
             const size_t minBufferSize = SensorEventQueue::MAX_RECEIVE_BUFFER_EVENT_COUNT;
             mSensorEventBuffer = new sensors_event_t[minBufferSize];
             mSensorEventScratch = new sensors_event_t[minBufferSize];
             mMapFlushEventsToConnections = new SensorEventConnection const * [minBufferSize];
+
             mInitCheck = NO_ERROR;
+            run("SensorService", PRIORITY_URGENT_DISPLAY);
         }
     }
 }