【Android】【armeabi-v7a】nativeReplaceImageByName JNI 参数类型错误导致 SIGSEGV 崩溃

[zhaozenghaozzh-ops]

您好，libpag 团队：

我们对线上 replaceImageByName() 崩溃进行了详细排查，确认这里可能存在一个 JNI 参数类型不匹配的问题，主要影响 armeabi-v7a 设备。

环境信息

• libpag 版本：4.5.52
• 平台：Android 原生
• ABI：armeabi-v7a
• libpag.so Build ID：
  d6c34773a18cbbc53e2ddf8f8cb4642790030cda
• Firebase Crashlytics：累计 3,014 次事件，影响 220 名用户
• 我们抽查了 50 条有效事件，其中 38 条都指向相同的 libpag 崩溃路径

崩溃特征

SIGSEGV / SEGV_MAPERR
部分事件的 fault address 为 0x9

Java_org_libpag_PAGFile_nativeReplaceImageByName + 0x51
org.libpag.PAGFile.replaceImageByName
初步确认的根因
Java 层将 PAGImage.nativeContext 作为 Java long 传入：
public native void nativeReplaceImageByName(
    String layerName,
    long image
);
但是对应的 C++ JNI 实现中，imageObject 使用的是 C++ long：
PAG_API void Java_org_libpag_PAGFile_nativeReplaceImageByName(
    JNIEnv* env,
    jobject thiz,
    jstring layerName,
    long imageObject
)
在 Android armeabi-v7a（LP32）环境中：
sizeof(long)  == 4
sizeof(jlong) == 8
Java long 是 64 位，但 ARMv7 下的 C++ long 只有 32 位。这会导致 Java 与 native 使用不兼容的 ARM 参数传递方式，JNI 函数可能读取到错误的 PAGImage 指针，最终产生 0x9 等非法地址访问。
同一个文件中，相邻的 nativeReplaceImage() 已经正确使用了 jlong：
PAG_API void Java_org_libpag_PAGFile_nativeReplaceImage(
    JNIEnv* env,
    jobject thiz,
    jint index,
    jlong imageObject
)
目前最新的 4.5.75 版本以及 main 分支中，nativeReplaceImageByName() 仍然使用 C++ long。
建议修复
 PAG_API void Java_org_libpag_PAGFile_nativeReplaceImageByName(
     JNIEnv* env,
     jobject thiz,
     jstring layerName,
-    long imageObject
+    jlong imageObject
 )
烦请官方帮忙确认这个 JNI ABI 问题，并考虑：
在 main 分支中将 long imageObject 修正为 jlong imageObject；
将修复回合到 Android 4.5.x 版本；
增加 armeabi-v7a 下调用 replaceImageByName() 的回归测试。
我们当前的临时规避方案是停止调用 replaceImageByName()，根据图层名解析 editable image index，然后改用参数类型正确的：
replaceImage(index, image)
相关源码：
https://github.com/Tencent/libpag/blob/v4.5.75/src/platform/android/JPAGFile.cpp#L170-L200
https://github.com/Tencent/libpag/blob/v4.5.75/android/libpag/src/main/java/org/libpag/PAGFile.java#L133-L159
感谢！

[CodeJhF]

感谢您的详细排查和反馈。我们已经关注到了这个问题，会在近期修复。

[CodeJhF]

补充一下：该修复已通过 PR #3594 合入 main，并已 cherry-pick 到 release/4.5 和 release/4.4 分支，将随下个版本发布。感谢详细的排查和反馈！

[zhaozenghaozzh-ops]

好的，感谢