ZLMediaKit MP4录制分片异常问题分析与修复

问题背景

在ZLMediaKit项目中，用户报告了一个关于MP4录制功能的异常现象。当使用RTSP拉流进行MP4录制时，如果设置了自动分片时间为10分钟，第一个切片录制正常，但从第二个切片开始，系统会异常地每2-3秒就生成一个新的切片文件。这个问题出现在2024年11月1日的代码更新后。

问题分析

经过开发团队深入排查，发现这个问题源于MP4Recorder.cpp文件中的一个逻辑错误。具体来说，在处理视频帧的时间戳(DTS)时存在缺陷，导致切片判断逻辑出现异常。

在视频编码中，DTS(Decoding Time Stamp)表示解码时间戳，而PTS(Presentation Time Stamp)表示显示时间戳。由于B帧的存在，DTS可能会出现回退现象，这是正常的视频编码特性。原代码在处理DTS回退时采用了MIN(frame->dts(), _last_dts)的方式，这在_last_dts为0时的初始状态下会导致问题。

问题根源

问题的核心在于时间戳处理的逻辑不够严谨。当_last_dts为0时（初始状态），直接使用MIN操作会导致_last_dts始终为0，因为任何正数的时间戳与0取最小值都会得到0。这破坏了时间戳的正常累计，进而导致切片判断逻辑失效。

具体表现为：

1. 第一个切片正常，因为初始状态_last_dts=0，可以正确记录第一个时间戳
2. 从第二个切片开始，由于_last_dts被错误地保持为0，系统会误判每个新帧的时间戳都远大于_last_dts，从而频繁触发切片条件

解决方案

开发团队提出了以下修复方案：

if (_last_dts == 0 || _last_dts > frame->dts()) {
    // b帧情况下dts时间戳可能回退
    if(_last_dts == 0){
        _last_dts = frame->dts();
    }else{
        _last_dts = MIN(frame->dts(), _last_dts);
    }
}

这个修复方案的关键改进是：

1. 区分初始状态(_last_dts==0)和非初始状态
2. 在初始状态下，直接使用当前帧的dts作为_last_dts
3. 在非初始状态下，才使用MIN操作来处理可能的DTS回退

技术启示

这个问题给我们几个重要的技术启示：

1. 时间戳处理在多媒体应用中至关重要，特别是涉及到B帧等复杂编码结构时
2. 初始状态的处理需要特别小心，不能简单地套用常规逻辑
3. MIN/MAX等边界操作在特殊情况下可能产生非预期结果
4. 录制功能的稳定性不仅影响用户体验，还可能影响后续的播放和处理

总结

ZLMediaKit作为一款优秀的流媒体服务器，其录制功能的稳定性对很多应用场景至关重要。这次问题的发现和修复体现了开源社区协作的优势，也展示了开发团队对问题快速响应和解决的能力。通过这个案例，我们不仅学习到了一个具体问题的解决方法，更理解了多媒体应用中时间戳处理的复杂性和重要性。