Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Prevent use of a null sCallbackObj by Hardware FLP" into nyc-dev

Browse Source 
am: 793ac34219

* commit '793ac342196c57f6b89ad9d1425fc420fbb44bb5':
  Prevent use of a null sCallbackObj by Hardware FLP

Change-Id: I3681a3433f21112659840fb4431f0f616305ee3c
This commit is contained in:
Wyatt Riley
2016-05-27 03:23:44 +00:00
committed by android-build-merger
parent d93090d916 793ac34219
commit 4e6cd80198
1 changed files with 16 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
services/core/jni/com_android_server_location_FlpHardwareProvider.cpp
View File

@@ -78,7 +78,7 @@ static inline void ThrowOnError(
env->ThrowNew(exceptionClass, methodName);
}
static bool IsValidCallbackThread() {
static bool IsValidCallbackThreadEnvOnly() {
JNIEnv* env = AndroidRuntime::getJNIEnv();
if(sCallbackEnv == NULL || sCallbackEnv != env) {
@@ -89,6 +89,20 @@ static bool IsValidCallbackThread() {
return true;
}
static bool IsValidCallbackThread() {
// sCallbacksObject is created when FlpHardwareProvider on Java side is
// initialized. Sometimes the hardware may call a function before the Java
// side is ready. In order to prevent a system crash, check whether
// sCallbacksObj has been created. If not, simply ignore this event from
// hardware.
if (sCallbacksObj == NULL) {
ALOGE("Attempt to use FlpHardwareProvider blocked, because it hasn't been initialized.");
return false;
}
return IsValidCallbackThreadEnvOnly();
}
static void BatchingCapabilitiesCallback(int32_t capabilities) {
if(!IsValidCallbackThread()) {
return;
@@ -154,7 +168,7 @@ static int SetThreadEvent(ThreadEvent event) {
}
case DISASSOCIATE_JVM:
{
if (!IsValidCallbackThread()) {
if (!IsValidCallbackThreadEnvOnly()) {
ALOGE(
"Attempted to dissasociate an unnownk callback thread : '%s'.",
__FUNCTION__
@@ -661,16 +675,6 @@ static void GeofenceMonitorStatusCallback(
TranslateToObject(lastLocation, locationObject);
}
// sCallbacksObject is created when FlpHardwareProvider on Java side is
// initialized. Sometimes the hardware may call this function before the Java
// side is ready. In order to prevent the system crash, check whether
// sCallbacksObj has been created. If not, simply ignore this event from
// hardware.
if (sCallbacksObj == NULL) {
ALOGE("FlpHardwareProvider hasn't been initialized.");
return;
}
sCallbackEnv->CallVoidMethod(
sCallbacksObj,
sOnGeofenceMonitorStatus,