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

问题背景

在SRS(Simple Realtime Server)流媒体服务器的release6.0版本中，用户发现了一个关于HTTP-FLV流处理的严重问题。当使用OBS推流并通过ffplay播放时，如果推流中断后立即重新推流，会出现"Failed to handle HTTP request for pattern duplicated"错误，导致后续推流一直失败，报"Stream already exists or busy"错误。

问题现象

具体表现为：

1. 使用OBS推流，同时用ffplay播放拉流
2. OBS断开推流后立即重新推流
3. 有概率出现HTTP请求处理失败的错误
4. 之后所有重新推流尝试都会失败，提示流已存在或繁忙

问题分析

通过分析日志和代码，发现问题出在SRS的流状态管理机制上。具体原因如下：

1. 流状态转换问题：当推流断开时，SrsLiveSource::on_unpublish()函数被调用，该函数过早地将_can_publish标志设为true，而此时流的清理工作尚未完成。

2. HTTP挂载点冲突：在流未完全清理的情况下，新的推流尝试会触发HTTP挂载点冲突，因为之前的挂载点还未释放。

3. 状态不一致：一旦出现挂载点冲突错误，流的_can_publish状态会被错误地锁定为false，导致后续所有推流尝试都被拒绝。

技术细节

问题的核心在于SrsLiveSource::on_unpublish()函数的执行顺序：

void SrsLiveSource::on_unpublish()
{
    _can_publish = true; // 过早设置
    
    // 执行各种清理操作...
    handler->on_unpublish(req); // 这里可能触发协程切换
    
    // 其他清理工作...
}

当handler->on_unpublish(req)被调用时，会触发HTTP卸载操作，这可能导致协程切换。如果在卸载完成前就有新的推流请求到来，就会导致挂载点冲突。

解决方案

修复方案是将_can_publish = true移到函数末尾，确保所有清理工作完成后再开放推流权限：

void SrsLiveSource::on_unpublish()
{
    // 执行各种清理操作...
    handler->on_unpublish(req);
    
    // 其他清理工作...
    
    _can_publish = true; // 确保所有清理完成后再开放推流
}

这个修改确保了：

1. 所有资源释放完成后再允许新的推流
2. 避免了HTTP挂载点的竞争条件
3. 保持了流状态的一致性

总结

这个问题的修复体现了流媒体服务器开发中几个重要原则：

1. 状态转换的原子性：关键状态变更应该在所有相关操作完成后进行
2. 资源管理的时序性：资源释放和重新分配需要严格的时间顺序
3. 异常处理的完备性：需要考虑所有可能的执行路径

对于SRS用户来说，这个修复确保了推流中断后能够正常恢复，提高了服务的可靠性。对于开发者而言，这个案例也提醒我们在设计状态机时需要仔细考虑所有可能的执行路径和时序问题。