Shell 重定向导致终端无限打印

Question

所以我正在用 C 语言编写一个可以执行 STDOUT 重定向的简单 shell。我正在从用户那里读取一个字符串，将其拆分为参数，然后将其提供给 exec。但是当我重定向输出时，它的第一个：创建我要求的文件并用正确的数据填充它，然后在终端中无限打印命令结果。我在 dup2 上做错了吗？

这是我的重定向代码。使用的所有变量都是全局变量，文件描述符除外。

int execRedirectCommand(){
    int fd;

    pid_t pid = fork();


    // fork failed
    if(pid == -1){
        char* error = strerror(errno);
        printf("fork: %s\n", error);
        return -1;
    }
    //Child process
    else if(pid == 0){
        fd = open(fileName, O_WRONLY | O_CREAT | O_TRUNC, 0755);
        if(fd == -1){
            char* error = strerror(errno);
            printf("open:%s\n", error);
            return -1;
        }
        dup2(fd, STDOUT_FILENO);
        close(fd);

        execvp(payloadArgv[0], payloadArgv);

        // execvp failed
        char* error = strerror(errno);
        printf("pdsh:%s:%s\n", payloadArgv[0], error);
        return -1;
    }
    // Parent process
    else{
        close(fd);
        fileName = NULL;

        // Wait for child process to finish
        int childStatus;
        waitpid(pid, &childStatus, 0);
        return 0;
    }
}

EDIT 修正了代码中的一个错字，没什么大不了的。 编辑 2 包括我的主要内容：

int main(){
    setSigHandler();
    char* user = getlogin();


    while(1){
        printf("[%s]-->$", user);
        getNextCommand(payload);

        if(!strcmp(payload, "\n")) continue;

        if(!strcmp(payload, "close")) break;


        parseCommandString();


        if(fileName != NULL){
            execRedirectCommand();
        }else{
            execSimpleCommand();
        }


    }

    return 0; }

注意：没有 STDOUT 重定向的命令可以正常工作。

Answer 1

感谢 mata 和 nsilent22 指出这一点。不需要父进程中的close(fd)。