Janus Gateway中SIP插件PeerConnection未正常关闭问题分析

在Janus Gateway最新主分支版本中，发现了一个与SIP插件PeerConnection(PC)关闭机制相关的问题。该问题会导致在某些特定场景下，服务器端的PeerConnection无法被正确关闭，最终只能依赖DTLS警报来终止连接。

问题背景

Janus Gateway是一个开源的WebRTC服务器，其SIP插件用于实现WebRTC与SIP协议之间的互通。在正常的通话挂断流程中，系统应当能够正确清理所有相关资源，包括关闭PeerConnection。

问题现象

当使用SIP插件时，如果客户端发送"hangup"请求但不主动关闭PeerConnection，会出现以下异常流程：

1. Janus将SIP会话标记为关闭状态
2. SIP中继线程循环终止，调用janus_sip_media_cleanup方法
3. 该方法将has_audio和has_video标志位设为FALSE
4. 随后触发的nua_i_state事件中，由于上述标志位已为FALSE，PeerConnection不会被关闭

这种异常情况会导致PeerConnection保持打开状态，最终只能通过DTLS警报来强制关闭连接。

技术分析

深入分析问题根源，发现这与Janus Gateway的一个关键提交有关。该提交修改了资源清理逻辑，但引入了一个时序问题：

• 在janus_sip_sofia_callback处理nua_i_state事件时，会检查has_audio和has_video标志位
• 由于janus_sip_media_cleanup已经将这些标志位置为FALSE，导致PeerConnection关闭逻辑被跳过
• 资源清理的顺序和时序变得至关重要

解决方案

经过讨论和测试，确定了以下修复方案：

1. 修改PeerConnection关闭逻辑，使其不再依赖has_audio和has_video标志位
2. 确保无论是否存在活跃的媒体流，都能正确关闭PeerConnection
3. 同时保持资源清理操作的安全性和幂等性

具体实现上，将原本的条件判断逻辑改为直接调用close_pc和janus_sip_hangup_media_internal方法，确保资源能够被正确释放。

最佳实践建议

对于使用Janus Gateway SIP插件的开发者，建议：

1. 客户端在发送"hangup"请求后，应主动关闭PeerConnection
2. 服务器端应确保在各种情况下都能正确清理资源
3. 注意处理异常情况下的资源释放，避免内存泄漏
4. 定期更新到最新版本，获取问题修复和性能改进

该问题的修复已经合并到Janus Gateway的主分支和稳定版本中，建议所有用户及时更新以获取更稳定的通话体验。