@arms/rum-reactnative 0.1.3-beta.0 → 0.1.3-beta.1

package/android/src/main/jni/rum_crash_writer.c

@@ -191,7 +191,7 @@ static void emit_frame(int fd, char *buf, size_t *pos,
 
     int si = rum_module_index_for_addr(addr);
     if (si >= 0) {
-        uintptr_t base = rum_seg_load_base(si);
+        uintptr_t base = rum_module_base_for_seg(si);
 
         // 记录被引用模块的 image_addr（去重），供 binary_images 过滤
         if (base != 0 && *referenced_count < MAX_REFERENCED_MODULES) {
@@ -316,6 +316,9 @@ void rum_write_crash(int fd, int signum, siginfo_t *info, ucontext_t *uc,
 
     // 解析 /proc/self/maps（async-signal-safe），为帧提供模块归属与相对偏移
     rum_modules_parse();
+    // 构建去重模块列表，供 emit_frame 获取模块级基址和 binary_images 枚举使用。
+    // 必须在 frames 循环前调用，确保 rum_module_base_for_seg() 能查到模块。
+    int mod_count = rum_module_build_distinct();
 
     // binary_images 按需过滤：仅保留被栈帧引用的模块。
     // 参照 Sentry 实践，服务端符号化时忽略未被任何栈帧引用的模块，
@@ -328,36 +331,28 @@ void rum_write_crash(int fd, int signum, siginfo_t *info, ucontext_t *uc,
     // "frames":[{"address","library","image_addr","offset"}, ...]
     // library/offset 供后端结合 binary_images 的 build_id 精确符号化
     //
-    // 双侧截断（参照 Sentry 推荐方案）：frame_count 达到 TRUNCATION_THRESHOLD(512)
-    // 时说明实际栈可能更长（信号处理器中只能采集前 MAX_FRAMES 帧）。此时保留
-    // 前 FRAMES_PER_SIDE 帧和后 FRAMES_PER_SIDE 帧，中间插入标记帧。
-    // 注意：NDK 信号处理器只能获取前 512 帧，无法获取真正的"后 256 帧"
-    // （靠近 main 的帧），omitted_count=-1 表示实际总帧数未知。
+    // NDK 信号处理器最多采集 MAX_FRAMES(512) 帧。frame_count 达到
+    // TRUNCATION_THRESHOLD(512) 时说明实际栈可能更长（已达到采集上限），
+    // 此时全部输出 512 帧 + 尾部标记帧（omitted_count=-1 表示实际总帧数
+    // 未知），不做双侧截断——因为 NDK 只能获取前 512 帧，前 256 + 后 256
+    // 恰好覆盖全部 512 帧，中间无省略，插入中间标记帧会造成冗余。
     // frame_count < TRUNCATION_THRESHOLD 时全部输出，不截断。
     // 所有操作仅为栈上数组读写与整数比较，符合 async-signal-safe。
     buf_append(fd, buf, &pos, ",\"frames\":[");
-    int needs_truncation = (frame_count >= TRUNCATION_THRESHOLD);
+    int needs_tail_marker = (frame_count >= TRUNCATION_THRESHOLD);
     int first = 1;
 
-    int head_count = needs_truncation ? FRAMES_PER_SIDE : frame_count;
-    for (int i = 0; i < head_count; i++) {
+    for (int i = 0; i < frame_count; i++) {
         emit_frame(fd, buf, &pos, (uintptr_t)frames[i],
                    referenced_addrs, &referenced_count, &first);
     }
 
-    if (needs_truncation) {
-        // 截断标记帧（omitted_count=-1 表示实际总帧数未知）
+    if (needs_tail_marker) {
+        // 尾部标记帧（omitted_count=-1 表示实际总帧数未知）
         if (!first) {
             buf_append(fd, buf, &pos, ",");
         }
         buf_append(fd, buf, &pos, "{\"truncated\":true,\"omitted_count\":-1}");
-        first = 0;
-
-        int tail_start = frame_count - FRAMES_PER_SIDE;
-        for (int i = tail_start; i < frame_count; i++) {
-            emit_frame(fd, buf, &pos, (uintptr_t)frames[i],
-                       referenced_addrs, &referenced_count, &first);
-        }
     }
     buf_append(fd, buf, &pos, "]");
 
@@ -375,7 +370,6 @@ void rum_write_crash(int fd, int signum, siginfo_t *info, ucontext_t *uc,
     // 仅上报带 GNU build-id 的 file-backed 模块（build-id 等价于 iOS image uuid，
     // 用于后端精确匹配符号文件）。
     buf_append(fd, buf, &pos, ",\"binary_images\":[");
-    int mod_count = rum_module_build_distinct();
     int emitted = 0;
     char bid[BUILD_ID_HEX_CAP];
     for (int m = 0; m < mod_count; m++) {

package/android/src/main/jni/rum_module_info.c

@@ -152,6 +152,27 @@ uintptr_t rum_seg_load_base(int seg_index) {
     return g_segs[seg_index].start - g_segs[seg_index].offset;
 }
 
+uintptr_t rum_module_base_for_seg(int seg_index) {
+    if (seg_index < 0 || seg_index >= g_seg_count) {
+        return 0;
+    }
+    // 查找该段所属模块（同路径），返回模块级基址（image_addr）。
+    // 与 binary_images 的 rum_module_image_addr() 一致，确保 emit_frame
+    // 收集的 referenced_addrs 与 binary_images 过滤比较的基址相同。
+    // 若 rum_module_build_distinct() 尚未调用（g_mod_count == 0），
+    // 回退到段级基址。
+    const char *path = g_segs[seg_index].path;
+    int path_len = g_segs[seg_index].path_len;
+    for (int m = 0; m < g_mod_count; m++) {
+        if (g_mods[m].path_len == path_len &&
+            memcmp(g_mods[m].path, path, (size_t)path_len) == 0) {
+            return g_mods[m].image_addr;
+        }
+    }
+    // Fallback: 段级基址
+    return g_segs[seg_index].start - g_segs[seg_index].offset;
+}
+
 const char *rum_seg_path(int seg_index, int *out_len) {
     if (seg_index < 0 || seg_index >= g_seg_count) {
         if (out_len) *out_len = 0;
@@ -296,6 +317,24 @@ int rum_module_build_id_hex(uintptr_t load_base, char *out, int out_cap) {
             const unsigned char *name = note + sizeof(ElfW(Nhdr));
             const unsigned char *desc = name + ((namesz + 3u) & ~3u);
 
+            // 边界检查：确保 name 区域不超出 note section。
+            // 损坏的 namesz 可导致 name 指针越界，memcmp 读取未映射内存
+            // 触发二次 SIGSEGV。
+            if (name + namesz > note_end) {
+                break;
+            }
+
+            // 边界检查：desc 起始地址（含对齐填充）不得超出 note section。
+            if (desc > note_end) {
+                break;
+            }
+
+            // 边界检查：将 descsz clamp 到实际可用范围，防止 desc[b] 循环
+            // 读取超出 note section 的内存，触发二次 SIGSEGV。
+            if (desc + descsz > note_end) {
+                descsz = (uint32_t)(note_end - desc);
+            }
+
             if (type == NT_GNU_BUILD_ID && namesz == 4 &&
                 memcmp(name, "GNU", 4) == 0) {
                 static const char hexc[] = "0123456789abcdef";

package/android/src/main/jni/rum_module_info.h

@@ -24,6 +24,10 @@ int rum_module_index_for_addr(uintptr_t addr);
 /** 段的加载基址（= start - file_offset）。索引非法返回 0。 */
 uintptr_t rum_seg_load_base(int seg_index);
 
+/** 段所属模块的加载基址（模块级 image_addr，与 binary_images 一致）。
+ *  需在 rum_module_build_distinct() 之后调用；未构建时回退到段级基址。 */
+uintptr_t rum_module_base_for_seg(int seg_index);
+
 /** 段对应的模块路径（指向内部缓冲，非 null 结尾）。out_len 输出长度。 */
 const char *rum_seg_path(int seg_index, int *out_len);
 

package/lib/commonjs/utils/crash.js

@@ -119,6 +119,9 @@ function formatStack(frames) {
     // 截断标记帧：双侧截断时插入的中间标记
     if (frame.truncated) {
       const count = frame.omitted_count ?? -1;
+      if (count < 0) {
+        return `#${index} ... frames omitted (unknown count) ...`;
+      }
       return `#${index} ... ${count} frames omitted ...`;
     }
     const library = frame.library || '<unknown>';

package/lib/commonjs/utils/platform.js

@@ -14,7 +14,7 @@ var _rumCore = require("@arms/rum-core");
 var _reactNative = require("react-native");
 var _NativeRumSdkReactnative = _interopRequireDefault(require("../NativeRumSdkReactnative"));
 function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
-const _pkg = {"version":"0.1.3-beta.0"};
+const _pkg = {"version":"0.1.3-beta.1"};
 const VERSION = exports.VERSION = _pkg.version;
 
 // eslint-disable-next-line @typescript-eslint/no-unused-vars

package/lib/module/utils/crash.js

@@ -114,6 +114,9 @@ function formatStack(frames) {
     // 截断标记帧：双侧截断时插入的中间标记
     if (frame.truncated) {
       const count = frame.omitted_count ?? -1;
+      if (count < 0) {
+        return `#${index} ... frames omitted (unknown count) ...`;
+      }
       return `#${index} ... ${count} frames omitted ...`;
     }
     const library = frame.library || '<unknown>';

package/lib/module/utils/platform.js

@@ -3,7 +3,7 @@
 import { AppType } from '@arms/rum-core';
 import { Dimensions, Platform } from 'react-native';
 import NativeRumSdkReactnative from "../NativeRumSdkReactnative.js";
-const _pkg = {"version":"0.1.3-beta.0"};
+const _pkg = {"version":"0.1.3-beta.1"};
 export const VERSION = _pkg.version;
 
 // eslint-disable-next-line @typescript-eslint/no-unused-vars

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arms/rum-reactnative",
-  "version": "0.1.3-beta.0",
+  "version": "0.1.3-beta.1",
   "description": "RUM SDK For ReactNative",
   "homepage": "https://help.aliyun.com/zh/arms/user-experience-monitoring/",
   "main": "./lib/commonjs/index.js",