无参数cd调用时chdir(getenv("HOME"))返回Bad address问题咨询

你的代码存在两个关键问题导致无参数cd时报错：

• 逻辑分支错误：无参数时，t->argv[1]为NULL，第一个if执行chdir(home)后，第二个独立的if会继续调用chdir(t->argv[1])，也就是chdir(NULL)，这会触发"Bad address"错误。
• 未处理HOME变量为空的情况：如果getenv("HOME")返回NULL（比如环境变量未正确设置），直接调用chdir(home)也会出错。

修正后的代码如下：

char *home = getenv("HOME");
if (!strcmp(t->argv[0], "cd")) {
    // 优先处理带参数的情况
    if (t->argv[1]) {
        if (chdir(t->argv[1])) {
            perror(t->argv[1]);
        }
    } else {
        // 无参数时，先检查HOME变量是否有效
        if (home) {
            if (chdir(home)) {
                perror(home);
            }
        } else {
            fprintf(stderr, "cd: HOME environment variable not set\n");
        }
    }
}

修改说明： #

1. 把两个独立的if改成if-else结构，确保无参数时不会执行带参数的分支，避免传入NULL给chdir。
2. 增加对home变量的非空检查，当HOME环境变量未设置时，给出明确的错误提示，而不是触发系统错误。
3. 调整分支顺序，先处理带参数的场景，逻辑更清晰。

这样修改后，无参数cd会正确切换到HOME目录，且不会触发错误；同时也兼容了HOME变量未设置的异常情况，保证多机器适配的需求。

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