ireader/media-server项目中SDP解析模块的IP地址获取问题分析

在流媒体服务器开发中，SDP(Session Description Protocol)协议扮演着重要角色，它用于描述多媒体会话的初始化参数。ireader/media-server作为一个功能完善的流媒体服务器项目，其SDP解析模块的实现质量直接影响着整个系统的稳定性和可靠性。

问题背景

在解析SDP数据时，connection字段包含了会话或媒体流的网络连接信息，其中最重要的就是IP地址。项目中的sdp_connection_get_address()函数负责从SDP的connection字段中提取IP地址，但原始实现存在一个指针操作错误。

问题分析

原始代码中的指针操作存在逻辑缺陷，具体表现为：

while(*p != '\0' && *p != '/' && *p != ' ' && *p != '\t' && *p != '\r' && *p != '\n') {
    *ip++ = *p;  // 错误：只复制字符但未移动源指针
}

这段代码的问题在于：

1. 只将源指针p指向的内容复制到目标ip指针
2. 但忘记同时递增源指针p
3. 导致无限循环或内存访问越界

正确的实现应该是：

while(*p != '\0' && *p != '/' && *p != ' ' && *p != '\t' && *p != '\r' && *p != '\n') {
    *ip++ = *p++;  // 正确：同时移动源指针和目标指针
}

影响范围

该问题影响以下两个关键函数：

1. sdp_connection_get_address() - 获取会话级connection的IP地址
2. sdp_media_get_connection_address() - 获取媒体级connection的IP地址

这两个函数在建立媒体会话时被频繁调用，错误的IP地址解析会导致：

• RTSP会话建立失败
• 媒体流无法正确传输
• 潜在的缓冲区溢出风险

解决方案

修复方案简单直接，只需在复制字符的同时递增源指针。这种修复：

1. 保持了原有函数接口不变
2. 不引入额外内存开销
3. 完全兼容现有调用方式

经验总结

这个案例提醒我们在处理字符串和指针操作时需要注意：

1. 指针递增的对称性 - 源指针和目标指针通常需要同步移动
2. 边界条件的检查 - 确保不会越界访问内存
3. 循环终止条件 - 避免因指针操作错误导致无限循环

在流媒体服务器这类对稳定性要求极高的系统中，即使是如此微小的指针操作错误也可能导致严重问题。开发者应当特别重视这类基础但关键的代码实现细节。