为何出现segmentation错误与malloc() corrupted topsize？附环境变量修改需求

错误原因分析 #

• 堆内存破坏：原代码手动计算环境变量长度时，只遍历了第一个环境变量（i<1），后续更长的环境变量用strcpy写入string时会越界，破坏malloc维护的堆结构，触发malloc() corrupted topsize错误。
• 环境变量表失效：setenv可能会重新分配系统环境变量表，原envp指针可能不再指向有效内存，后续遍历会引发段错误。
• 字符串终止符判断错误：用NULL（指针类型）判断字符终止符，正确的应该是'\0'（字符类型），错误的判断可能导致循环异常。

修复后的代码 #

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int main(int argc, char **argv, char **envp)
{
    // 先打印原始环境变量
    for(int i=0; envp[i]!=NULL; i++)
        printf("%d.%s\n", i , envp[i]);
    printf("-----------------------------------\n");

    // 遍历所有环境变量，处理偶数位置项
    for(int i=0; envp[i]!=NULL; i++)
    {
        // 自动分配内存复制当前环境变量字符串，避免手动计算长度出错
        char *string = strdup(envp[i]);
        if(string == NULL) {
            perror("strdup failed");
            exit(EXIT_FAILURE);
        }

        // 快速定位等号，分割变量名和值
        char *eq_pos = strchr(string, '=');
        if(eq_pos != NULL) {
            *eq_pos = '\0';
            // 偶数位置（从0开始计数）设置值为"LONDON"
            if(i % 2 == 0) {
                setenv(string, "LONDON", 1);
            }
        }
        // 释放strdup分配的内存，避免泄漏
        free(string);
    }

    // 使用系统全局环境变量表获取最新内容
    extern char **environ;
    for(int i=0; environ[i]!=NULL; i++)
        printf("%d.%s\n", i , environ[i]);

    return 0;
}

关键修复说明 #

• 用strdup替代手动malloc：strdup会自动计算字符串长度并分配足够内存，彻底避免内存越界问题。
• 使用全局environ变量：setenv修改环境后，原envp可能失效，environ是系统维护的全局指针，始终指向最新的环境变量表。
• 简化变量名分割逻辑：用strchr直接定位等号，比手动循环更高效可靠。
• 及时释放内存：strdup分配的内存用完后立即free，避免内存泄漏。

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