Merge "Add check for valid AIBinder_Class_setTransactionCodeToFunctionNameMap" into main

waghpawan · Gerrit Code Review · commit 4811398ed12a · 2024-10-10T20:02:34.000Z
diff --git a/libs/binder/ndk/include_cpp/android/binder_interface_utils.h b/libs/binder/ndk/include_cpp/android/binder_interface_utils.h
@@ -299,7 +299,8 @@ AIBinder_Class* ICInterface::defineClass(const char* interfaceDescriptor,
 
 #if defined(__ANDROID_UNAVAILABLE_SYMBOLS_ARE_WEAK__) || __ANDROID_API__ >= 36
     if API_LEVEL_AT_LEAST (36, 202504) {
-        if (codeToFunction != nullptr) {
+        if (codeToFunction != nullptr &&
+            (&AIBinder_Class_setTransactionCodeToFunctionNameMap != nullptr)) {
             AIBinder_Class_setTransactionCodeToFunctionNameMap(clazz, codeToFunction,
                                                                functionCount);
         }

Original file line number	Diff line number	Diff line change
`@@ -299,7 +299,8 @@ AIBinder_Class* ICInterface::defineClass(const char* interfaceDescriptor,`
`299`	`299`
`300`	`300`	`#if defined(__ANDROID_UNAVAILABLE_SYMBOLS_ARE_WEAK__) \|\| __ANDROID_API__ >= 36`
`301`	`301`	`if API_LEVEL_AT_LEAST (36, 202504) {`
`302`		`- if (codeToFunction != nullptr) {`
	`302`	`+ if (codeToFunction != nullptr &&`
	`303`	`+ (&AIBinder_Class_setTransactionCodeToFunctionNameMap != nullptr)) {`
`303`	`304`	`AIBinder_Class_setTransactionCodeToFunctionNameMap(clazz, codeToFunction,`
`304`	`305`	`functionCount);`
`305`	`306`	`}`