From 7830ef3dd0ff3749d974c2dd85a8fa59dc47aeca Mon Sep 17 00:00:00 2001 From: Aravind Akella Date: Tue, 7 Oct 2014 14:13:12 -0700 Subject: [PATCH] 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 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/services/sensorservice/SensorService.cpp b/services/sensorservice/SensorService.cpp index 8831893b0..0f945f765 100644 --- a/services/sensorservice/SensorService.cpp +++ b/services/sensorservice/SensorService.cpp @@ -185,14 +185,14 @@ void SensorService::onFirstRef() } 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); } } }