能否直接对open_memstream返回的request_buffer调用strtok解析？

背景说明 #

你正在实现支持HTTP/1.1流水线请求的服务器，使用open_memstream动态申请缓冲区存储客户端请求数据，需要在不复制缓冲区的前提下解析请求头，担心strtok_r修改缓冲区影响open_memstream运行。
你的原始接收数据代码如下：

char *request_buffer;
size_t request_bytes = 0, recv_bytes;
FILE *request_stream = open_memstream(&request_buffer, &request_bytes);

bool headers_received = false;

while (!headers_received) {
    char buf[BUFSIZ] = {0};
    recv_bytes = recv(client_socketfd, buf, BUFSIZ - 1, NULL);
    if (recv_bytes == -1) {
        perror("recv");
        continue;
    } else if (recv_bytes == 0) {
        printf("Connection closed\n");
        break;
    }
    fprintf(request_stream, "%s", buf);
    if (strstr(buf, "\r\n\r\n") != NULL)
        headers_received = true;
}

你纠结的两个解析方案中，方案二（直接解析request_buffer）是可行的，不需要额外复制缓冲区浪费内存。 #

可行性说明 #

直接对request_buffer使用strtok_r解析完全合法，不会干扰open_memstream的正常运行，原因如下：

• POSIX标准对open_memstream的规定是：只有调用fflush()、fseek()/fseeko()、fclose()时，才会更新对外暴露的request_buffer指针和request_bytes长度，刷新后的缓冲区内容是合法的、以'\0'结尾的，你可以对这块内存做任意合法修改。
• strtok_r的修改逻辑只是将匹配到的分隔符替换为'\0'，属于普通的用户态内存写入，open_memstream的内部实现只会维护写入偏移、在偏移位置追加新数据，不会校验或依赖已写入区域的原有内容，所以二者完全不会冲突。
• 加上你采用每个客户端fork单独进程的处理模式，不存在多线程并发访问流和缓冲区的问题，所以方案二的安全性完全可以保证。

需修复的代码问题 #

你当前的实现存在两个隐含bug，需要修改后才能稳定运行：

1. 访问缓冲区前需要先刷新流
   你在调用fprintf写入流之后，没有执行fflush(request_stream)就直接访问request_buffer，此时request_buffer的内容和request_bytes的取值可能不是最新的，会导致解析到不完整的数据。只需要在退出接收循环后加一行fflush(request_stream)即可。
2. 请求头结束标记匹配逻辑有边界问题
   你当前只在单次接收的buf中匹配\r\n\r\n，如果结束标记刚好跨两次接收的缓冲区边界（比如第一次接收的最后两个字节是\r\n，第二次接收的前两个字节是\r\n），就会匹配失败导致无限循环收数据。需要每次接收刷新流后，在整个request_buffer中匹配结束标记，或者单独维护最后几个字节的滑动窗口做匹配。

流水线请求适配建议 #

如果要支持HTTP/1.1流水线请求，解析完当前请求后需要计算该请求总共占用的字节数（请求行+请求头+请求体长度），剩余留在缓冲区的字节就是下一个流水线请求的开头，后续接收新数据直接追加到流后即可，不需要重置缓冲区。

内容的提问来源于stack exchange，提问作者user401445