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

Merge changes I9ad74037,I50b29ef9

Browse Source 
* changes:
  Avoid interface name copy in HwParcel
  Avoid allocation on Java Binder fast path
This commit is contained in:
Treehugger Robot
2020-02-26 14:46:57 +00:00
committed by Gerrit Code Review
parent 3e6b337bf4 74c60fbacc
commit c8a2dbdb10
2 changed files with 5 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
core/jni/android_os_HwParcel.cpp
View File

@@ -292,19 +292,11 @@ static void JHwParcel_native_enforceInterface(
return;
}
const jchar *interfaceName = env->GetStringCritical(interfaceNameObj, NULL);
const char *interfaceName = env->GetStringUTFChars(interfaceNameObj, NULL);
if (interfaceName) {
String8 interfaceNameCopy = String8(String16(
reinterpret_cast<const char16_t *>(interfaceName),
env->GetStringLength(interfaceNameObj)));
env->ReleaseStringCritical(interfaceNameObj, interfaceName);
interfaceName = NULL;
hardware::Parcel *parcel =
JHwParcel::GetNativeContext(env, thiz)->getParcel();
bool valid = parcel->enforceInterface(interfaceNameCopy.string());
bool valid = parcel->enforceInterface(interfaceName);
if (!valid) {
jniThrowException(
@@ -312,6 +304,7 @@ static void JHwParcel_native_enforceInterface(
"java/lang/SecurityException",
"HWBinder invocation to an incorrect interface");
}
env->ReleaseStringUTFChars(interfaceNameObj, interfaceName);
}
}

4
core/jni/android_os_Parcel.cpp
View File

@@ -625,8 +625,8 @@ static void android_os_Parcel_enforceInterface(JNIEnv* env, jclass clazz, jlong
IPCThreadState* threadState = IPCThreadState::self();
const int32_t oldPolicy = threadState->getStrictModePolicy();
const bool isValid = parcel->enforceInterface(
String16(reinterpret_cast<const char16_t*>(str),
env->GetStringLength(name)),
reinterpret_cast<const char16_t*>(str),
env->GetStringLength(name),
threadState);
env->ReleaseStringCritical(name, str);
if (isValid) {