NodeJS原生模块重复调用引发进程崩溃的问题排查及修复方案咨询

看起来你遇到的重复调用原生模块导致Electron进程崩溃的问题，核心原因藏在你的C++代码里的未初始化指针非法内存访问，加上重复调用时的内存错误累积，最终触发了崩溃。咱们一步步拆解问题和修复方案：

一、核心崩溃原因分析 #

先看你C++代码里EnumChildProc函数中的这段代码：

LPDWORD pid;
GetWindowThreadProcessId(hWnd, (LPDWORD)pid);

这里的pid是一个未初始化的LPDWORD指针，它指向的是随机的内存地址。GetWindowThreadProcessId会尝试往这个随机地址写入进程ID，这属于严重的未定义行为——第一次调用可能侥幸没触发崩溃，但重复调用时，这个非法内存操作很可能覆盖了进程关键内存区域，直接导致Electron进程崩溃（也就是你看到的crashpad_client_win.cc错误和退出码-36861）。

除此之外，还有两个潜在风险点：

• ContextData中的cb_是Function的引用，虽然当前EnumChildWindows是同步执行的，但如果后续改成异步逻辑，局部Function的生命周期结束会导致引用失效；
• 重复调用SetParent时，若同一个子窗口被多次设置父窗口，可能引发窗口句柄的资源冲突（不过这个是次要问题，先解决内存访问错误）。

二、具体修复方案 #

1. 修复未初始化指针的致命错误 #

修改EnumChildProc里的进程ID获取逻辑，用局部变量存储进程ID，再取地址传给API：

static BOOL CALLBACK EnumChildProc(HWND hWnd, LPARAM callbackLParam)
{
    ContextData *lParam = (ContextData *)callbackLParam;
    DWORD pid; // 用局部变量存储，避免未初始化指针
    GetWindowThreadProcessId(hWnd, &pid); // 传局部变量的地址
    uint32_t childWindowPid = pid;
    printf("childWindowPid is: %d\n", childWindowPid);
    uint32_t targetWindowPid = *(uint32_t *)(lParam->lpdwProcessId());
    printf("targetWindowPid is: %d\n", targetWindowPid);

    if (childWindowPid == targetWindowPid)
    {
        printf("find successfully\n");
        SetParent(hWnd, lParam->hWnd());
        (lParam->cb()).Call({});
        return FALSE;
    }
    return TRUE;
}

2. 优化ContextData的生命周期与引用安全 #

把ContextData中的cb_从引用改成值存储，避免局部Function引用失效的风险：

class ContextData
{
private:
    HWND hWnd_;
    LPDWORD lpdwProcessId_;
    Function cb_; // 改成值存储，而非引用
public:
    ContextData(Function cb, HWND hWnd, LPDWORD lpdwProcessId) : cb_(std::move(cb)), hWnd_(hWnd), lpdwProcessId_(lpdwProcessId) {}
    // ... 其他方法保持不变，cb()返回值即可
    Function& cb()
    {
        return cb_;
    }
};

同时，在Method函数中传递cb的时候，直接传值：

ContextData context(cb, hWnd, lpdwProcessId);

3. 避免重复调用时的窗口重复绑定 #

在JS侧，即使不调用controller.abort()，也应该避免重复对同一个子窗口执行SetParent。可以在找到目标窗口并完成绑定后，标记状态，后续循环中跳过执行：

const controller = new AbortController()
let isParentSet = false // 标记是否已完成绑定
try {
  for await (const _value of setInterval(5000, undefined, {
    signal: controller.signal,
  })) {
    if (isParentSet) continue // 已完成绑定则跳过
    SetParentNapi(buffer, pid, () => {
      log4js.info('SetParent success！')
      isParentSet = true
      // controller.abort() // 即使不abort，也通过状态跳过重复调用
    })
  }
} catch (error) {
  if (controller.signal.aborted) {
    log4js.debug('timeout aborted！')
    return
  }
  log4js.error(error)
}

三、验证修复效果 #

修复完C++代码后重新编译原生模块，然后运行JS代码：

• 即使不调用controller.abort()，重复执行时也不会再触发非法内存访问导致的崩溃；
• 通过JS侧的状态标记，可以避免不必要的重复窗口绑定操作，减少资源消耗。

备注：内容来源于stack exchange，提问作者Yao Tao