未初始化len指针会引发问题吗？GCC编译报错原因咨询

问题场景 #

声明了int* len;但未初始化，直接传给splitCommand函数，函数内部执行*len = i;赋值操作。用GCC加上-Wall -Werror编译时触发报错，导致编译终止。

相关代码 #

主函数中的调用代码：

int* len;
if (splitCommand(cmd_buff, commands, len) != 0){
    printError();
}

splitCommand函数实现：

int splitCommand(char* line, char** commands, int *len){
    char* command = strtok(line, ";&");

    int i = 0;
    while(command != NULL){
        commands[i] = command;
        command = strtok(NULL, ";&");
        i++;
    } 
    *len = i;
    return 0; 
}

编译报错信息 #

gcc -Wall -Werror -o myshell myshell.c
myshell.c: In function ‘main’:
myshell.c:131:13: error: ‘len’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
131 | if (splitCommand(cmd_buff, commands, len) != 0){
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make: *** [Makefile:7: myshell] Error 1

报错原因 #

int* len;只是声明了一个指针变量，但这个指针没有被初始化，它的取值是内存中的随机垃圾值——也就是说，它没有指向任何一块合法、可写的内存区域。

当splitCommand函数执行*len = i;时，本质是要往这个随机的内存地址写入数据，这属于C语言中的未定义行为（可能导致程序崩溃、数据损坏等）。GCC的-Wall选项会检测到这种危险的未初始化指针使用，而-Werror会把所有警告强制转为错误，直接终止编译流程。

解决办法 #

有两种常用的修复方式，核心都是让指针指向一块有效的内存：

方式1：使用栈上的普通int变量，传递它的地址 #

直接声明一个栈上的int变量，用&取它的地址传给函数，这样函数就能合法地往这个变量的内存地址写值：

// 声明栈上的int变量，拥有合法的内存地址
int len;
if (splitCommand(cmd_buff, commands, &len) != 0){
    printError();
}
// 后续可以直接使用len的值，比如打印命令数量
// printf("解析得到%d条命令\n", len);

方式2：动态分配内存给指针 #

如果确实需要用指针类型，可以通过malloc分配一块int大小的内存，让指针指向这块合法区域，注意使用完要释放内存避免泄漏：

// 分配一块int大小的内存，让指针指向它
int* len = malloc(sizeof(int));
// 必须检查内存分配是否成功
if (len == NULL) {
    perror("内存分配失败");
    exit(EXIT_FAILURE);
}
if (splitCommand(cmd_buff, commands, len) != 0){
    printError();
}
// 使用指针指向的值
// printf("解析得到%d条命令\n", *len);
// 用完释放内存，避免内存泄漏
free(len);
len = NULL; // 将指针置空，防止变成野指针

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