PJProject中Android相机处理线程安全关闭问题分析

问题背景

在PJProject项目的Android视频通话功能实现中，使用PJCamera2类(pjsua2)进行相机操作时，存在一个潜在的线程安全问题。当调用Stop()方法关闭相机时，处理线程(HandlerThread)被立即终止，而此时Camera2的回调类可能尚未完成关闭操作，导致系统抛出"Handler sending message to a dead thread"异常。

问题现象

开发者在停止视频通话时，logcat中会频繁出现如下错误日志：

Handler (android.os.Handler) {4d70809} sending message to a Handler on a dead thread
java.lang.IllegalStateException: Handler (android.os.Handler) {4d70809} sending message to a Handler on a dead thread

技术分析

当前实现的问题

1. 线程生命周期管理不当：PJCamera2的Stop()方法直接调用了handlerThread.quitSafely()并尝试join线程，但此时Camera2的回调可能仍在处理中。

2. 事件处理时序问题：Android Camera2 API采用异步回调机制，Stop()方法执行后，系统仍可能通过Handler发送回调事件。

3. 资源释放顺序：当前实现中线程被提前终止，导致后续的相机关闭回调无法正常处理。

问题根源

根本原因在于HandlerThread的生命周期管理没有与Camera2 API的回调机制正确同步。当CameraDevice.StateCallback的onClosed回调被触发时，处理线程可能已经被终止。

解决方案

改进方案

1. 延迟线程终止：将HandlerThread的终止操作移至CameraDevice.StateCallback的onClosed回调中。

2. 确保回调完成：只有在确认相机设备完全关闭后，才安全地终止处理线程。

3. 简化线程管理：移除Stop()方法中的线程join操作，改为完全依赖回调机制。

代码实现建议

private final CameraDevice.StateCallback camStateCallback = new CameraDevice.StateCallback() {
    @Override
    public void onClosed(CameraDevice c) {
        App.log(TAG, "CameraDevice.StateCallback.onClosed");
        // 安全终止处理线程
        if (handlerThread != null) {
            handlerThread.quitSafely();
            handlerThread = null;
            handler = null;
        }
    }
    // ... 其他回调方法
};

public void Stop() {
    // ... 其他停止逻辑
    // 移除原有的线程终止代码
}

技术验证

开发者已对该方案进行了充分测试：

1. 稳定性测试：经过100+次相机前后切换测试，验证了方案的稳定性。

2. 兼容性测试：在Android 9至15多个版本上验证通过。

3. 资源泄漏检查：确认不会因线程管理不当导致资源泄漏。

影响评估

虽然该问题不会导致PJSIP核心功能失效，但会带来以下影响：

1. 日志污染：频繁的错误日志会影响问题排查。

2. 潜在稳定性风险：在极端情况下可能导致不可预知的行为。

3. 用户体验：可能影响相机设备的快速切换。

最佳实践建议

1. 异步资源释放：对于依赖回调的API，资源释放操作应放在相应的回调中。

2. 线程生命周期管理：确保工作线程的生命周期覆盖所有可能的回调。

3. 错误处理：对于相机操作，应添加适当的错误恢复机制。

该改进方案已准备提交PR，将有效解决Android平台上PJProject视频通话的相机关闭问题。