Как в C перенаправить STDOUT_FILENO в /dev/null с помощью dup2, а затем перенаправить обратно к исходному значению позже?

У меня есть задание, над которым я работаю, и мне трудно его закончить. Идея состоит в том, чтобы написать программу if.c, которая выполняет одну программу и в случае успеха выполняет вторую программу. Я должен подавить стандартный вывод первой программы и восстановить стандартный вывод для второй. Я получаю сообщение об ошибке в нескольких тестах. Например: "./if echo no then echo yes" возвращает "echo:ошибка записи: неверный файловый дескриптор". Я пытался найти, что я делаю неправильно в Интернете, но не повезло.

Вот мой код:

#include <fcntl.h>
#include <sys/wait.h>
#include <stdio.h>
#include "tlpi_hdr.h"

int main(int argc, char *argv[])
{
    if(argc < 4){
        fprintf(stderr,"Incorrect number of arguments.\n");
        exit(EXIT_FAILURE);
    }

    int thenArg = 0;
    char then[4];
    strcpy(then,"then");
    for(int x=1; x<argc; x++){
        if(strncmp(argv[x], then, 4) == 0) thenArg = x;
    }

    if(thenArg == 0){
        fprintf(stderr,"No 'then' argument found.\n");
        exit(EXIT_FAILURE);
    }

    int save_out = dup(STDOUT_FILENO);
    if(save_out == -1){
        fprintf(stderr,"Error in dup(STDOUT_FILENO)\n");
        exit(EXIT_FAILURE);
    }

    int devNull = open("/dev/null",0);
    if(devNull == -1){
        fprintf(stderr,"Error in open('/dev/null',0)\n");
        exit(EXIT_FAILURE);
    }

    int dup2Result = dup2(devNull, STDOUT_FILENO);
    if(dup2Result == -1) {
        fprintf(stderr,"Error in dup2(devNull, STDOUT_FILENO)\n");
        exit(EXIT_FAILURE);
    }

    int program1argLocation = 1;
    int program2argLocation = thenArg + 1;
    int program1argCount = thenArg-1;
    int program2argCount = argc-(program2argLocation);
    char *program1args[program1argCount+1];
    char *program2args[program2argCount+1];

    for(int i=0; i<program1argCount; i++){
        program1args[i]=argv[program1argLocation + i];
    }
    program1args[program1argCount] = NULL;
    for(int i=0; i<program2argCount; i++){
        program2args[i]=argv[program2argLocation + i];
    }
    program2args[program2argCount] = NULL;

    pid_t pid = fork();
    int child_status;
    switch (pid) {
    case -1:
        fprintf(stderr,"Fork failed\n");
        exit(EXIT_FAILURE);

    case 0: //child
        //child will run program 1
        if(execvp(program1args[0],&program1args[0]) == -1){
            fprintf(stderr,"Program 1 Failed.\n");
            exit(EXIT_FAILURE);
        }  

    default: //parent
        //parent will run program2
        pid = wait(&child_status);

        if(WEXITSTATUS(child_status) == 0){
            dup2(save_out, STDOUT_FILENO);

            int prog2status = execvp(program2args[0],&program2args[0]);
            if(prog2status == -1) {
                fprintf(stderr,"Program 2 failed.\n");
                exit(EXIT_FAILURE);
            }  
        }  
    }  

}  

person Frank    schedule 13.02.2013    source источник
comment
char then[4]; strcpy(then, "then"); — это переполнение буфера. Почему бы вам просто не использовать if(0 == strcmp(argv[x], "then")) или if(0 == strncmp(argv[x], "then", 4)) вместо этого?   -  person nneonneo    schedule 13.02.2013
comment
Я думаю, что попробовал это в первый раз, когда реализовал это, и столкнулся с какой-то ошибкой. Я запомню это и вернусь к нему в следующий раз, когда столкнусь с тем же самым. Благодарю вас! Еще не привык к C, поэтому любой ввод, подобный этому, великолепен.   -  person Frank    schedule 13.02.2013


Ответы (1)


Ваша ошибка здесь:

int devNull = open("/dev/null",0);

Чтобы использовать devNull как STDOUT_FILENO, его нужно открыть для записи:

int devNull = open("/dev/null", O_WRONLY);
person caf    schedule 13.02.2013