libwebsockets CGI环境变量处理问题分析与修复

问题背景

在libwebsockets项目中，当使用其CGI服务器功能时，发现所有CGI环境变量的值都意外地以等号"="开头。例如，PATH环境变量的值变成了"=/bin:/usr/bin..."，而不是正常的"/bin:/usr/bin..."。这种异常现象会影响CGI脚本对环境的正确解析。

问题分析

通过查看libwebsockets的源代码，发现问题出在CGI环境变量处理逻辑中。在设置环境变量时，代码错误地将等号"="本身也包含在了变量值中。

关键问题代码段如下：

while (i->env_array[m]) {
    const char *p = strchr(i->env_array[m], '=');
    int naml = lws_ptr_diff(p, i->env_array[m]);
    char enam[32];

    lws_strnncpy(enam, i->env_array[m], naml, sizeof(enam));
    setenv(enam, p, 1);  // 这里直接将包含等号的指针传入
    m++;
}

技术原理

在CGI规范中，环境变量的格式应为"NAME=value"，其中：

• 等号"="是分隔符
• 等号前是变量名
• 等号后是变量值

libwebsockets在处理环境变量时，首先使用strchr()找到等号的位置，但在后续setenv()调用时，错误地将包含等号的指针直接作为值传入，导致等号被包含在最终的环境变量值中。

解决方案

修复方案非常简单，只需在设置环境变量值时跳过等号字符。修改后的代码如下：

while (i->env_array[m]) {
    const char *p = strchr(i->env_array[m], '=');
    int naml = lws_ptr_diff(p, i->env_array[m]);
    char enam[32];

    lws_strnncpy(enam, i->env_array[m], naml, sizeof(enam));
    setenv(enam, ++p, 1);  // 使用++p跳过等号
    m++;
}

影响范围

该问题影响libwebsockets 4.3.3版本及之前的main分支代码。修复已同时提交到main分支和v4-3-stable分支。

技术启示

这个案例展示了几个重要的编程实践：

1. 字符串处理时要特别注意分隔符的位置
2. 环境变量操作需要严格遵守规范格式
3. 简单的指针算术错误可能导致不符合预期的行为
4. 测试时应该验证环境变量的完整格式而不仅仅是功能性

对于CGI服务器开发者来说，正确处理环境变量是确保脚本能够正确解析请求信息的基础，这类看似简单的问题可能会在实际部署中造成难以排查的兼容性问题。