SRS开源流媒体服务器HTTP-FLV推流重连问题分析与修复

问题背景

在SRS(Simple-RTMP-Server)开源流媒体服务器的使用过程中，开发者发现了一个关于HTTP-FLV协议推流重连的严重问题。当使用OBS推流并通过ffplay播放时，如果推流中断后立即重新推流，会导致后续推流持续失败，并伴随服务器崩溃的风险。

问题现象

具体表现为：

1. 使用OBS推流，同时通过ffplay播放
2. 主动断开OBS推流后立即重新推流
3. 服务器日志出现"Failed to handle HTTP request for pattern duplicated"错误
4. 后续所有推流尝试都会失败，报错"Stream already exists or busy"
5. 严重情况下会导致服务器进程崩溃

问题根源分析

通过分析SRS源码和日志，发现问题出在流状态管理机制上。当推流断开时，SrsLiveSource::on_unpublish()方法会被调用，该方法中设置_can_publish = true的时机不当。

关键问题点：

1. on_unpublish()方法过早设置_can_publish = true
2. 后续的handler回调会触发http_unmount操作
3. http_unmount操作可能导致协程上下文切换
4. 这种执行顺序导致流状态管理出现竞态条件

技术细节

在SRS的源码实现中，SrsLiveSource类负责管理单个直播流的生命周期。其中_can_publish标志位控制是否允许新的推流连接。

原问题代码逻辑：

void SrsLiveSource::on_unpublish() {
    _can_publish = true;  // 过早设置可发布标志
    handler->on_unpublish(req);  // 触发http_unmount等操作
    // 其他清理逻辑...
}

这种实现会导致：

1. 在流还未完全清理完成时就允许新推流
2. 新旧流状态可能产生冲突
3. HTTP FLV挂载点可能未被正确释放
4. 最终导致流被标记为"busy"状态无法恢复

解决方案

修复方案非常简单但有效 - 将_can_publish = true的设置移到函数末尾：

void SrsLiveSource::on_unpublish() {
    // 先执行所有清理操作
    handler->on_unpublish(req);
    // 其他清理逻辑...
    
    _can_publish = true;  // 最后设置可发布标志
}

这个修改确保：

1. 所有资源清理完成后再开放新的推流
2. 避免状态管理竞态条件
3. 保证HTTP FLV挂载点正确释放
4. 流状态转换更加可靠

问题影响

该问题会影响所有使用HTTP-FLV协议的场景，特别是：

1. 需要频繁重连推流的应用场景
2. 网络不稳定的推流环境
3. 需要自动化推流管理的系统

修复后，SRS服务器能够正确处理HTTP-FLV推流的重连情况，提高了服务的稳定性和可靠性。

总结

这个案例展示了流媒体服务器中状态管理的重要性。通过对关键标志位设置时机的调整，解决了推流重连的核心问题。这也提醒开发者在设计状态转换逻辑时，需要考虑操作顺序对系统稳定性的影响，特别是在涉及网络I/O和协程切换的复杂场景中。