MiPad4/android_device_xiaomi_sdm660-common
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge "Check Null on call to LocationAPI::addGeofence"

Browse source
This commit is contained in:
Linux Build Service Account 2017-03-24 06:31:25 -07:00 committed by Gerrit - the friendly Code Review server
commit 8f6ba9dbc3
2 changed files with 13 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
android/location_api/GeofenceAPIClient.cpp
View file

@ -100,7 +100,10 @@ void GeofenceAPIClient::geofenceAdd(uint32_t geofence_id, double latitude, doubl
data.longitude = longitude; data.longitude = longitude;
data.radius = radius_meters; data.radius = radius_meters;
locAPIAddGeofences(1, &geofence_id, &options, &data); LocationError err = (LocationError)locAPIAddGeofences(1, &geofence_id, &options, &data);
if (LOCATION_ERROR_SUCCESS != err) {
onAddGeofencesCb(1, &err, &geofence_id);
}
} }
void GeofenceAPIClient::geofencePause(uint32_t geofence_id) void GeofenceAPIClient::geofencePause(uint32_t geofence_id)

17
location/LocationAPIClientBase.cpp
View file

@ -409,17 +409,18 @@ uint32_t LocationAPIClientBase::locAPIAddGeofences(
delete requests; delete requests;
} }
uint32_t* sessions = mLocationAPI->addGeofences(count, options, data); uint32_t* sessions = mLocationAPI->addGeofences(count, options, data);
LOC_LOGI("%s:%d] start new sessions: %p", __FUNCTION__, __LINE__, sessions); if (sessions) {
requests = new RequestQueue(-1); LOC_LOGI("%s:%d] start new sessions: %p", __FUNCTION__, __LINE__, sessions);
requests->push(new AddGeofencesRequest(*this)); requests = new RequestQueue(-1);
mRequestQueues[REQUEST_GEOFENCE] = requests; requests->push(new AddGeofencesRequest(*this));
mRequestQueues[REQUEST_GEOFENCE] = requests;
for (size_t i = 0; i < count; i++) { for (size_t i = 0; i < count; i++) {
mGeofenceBiDict.set(ids[i], sessions[i], options[i].breachTypeMask); mGeofenceBiDict.set(ids[i], sessions[i], options[i].breachTypeMask);
}
retVal = LOCATION_ERROR_SUCCESS;
} }
pthread_mutex_unlock(&mMutex); pthread_mutex_unlock(&mMutex);
retVal = LOCATION_ERROR_SUCCESS;
} }
return retVal; return retVal;