Cでリダイレクトを作成したい。

前提・実現したいこと

c言語でリダイレクトを作りたいのですが、なかなかうまく行きません。
初心者ですので簡単なミスかもしれませんが、結構考えて答えにたどり着かなかったので質問させていただきます。

該当のソースコード

コード
#include<unistd.h>
#include<string.h>
#include<fcntl.h>
#include<sys/types.h>
#include<sys/wait.h>

#define MAX_LINE_IN 1000
#define MAX_ARGS 30


void redirect(char *args[], int REDI){
  int fd;
//  printf("%sこれにリダイレクトします。\n", args[REDI]);
                                                                                                                                                                                 
  fd=open(args[REDI], O_WRONLY | O_CREAT, 0664);
  close(STDOUT_FILENO);
  dup(fd);

  execvp(args[0],args);
}

int main(int argc, char *argv[]){
  int pid, status;
  char line_in[MAX_LINE_IN];
  char *args[MAX_ARGS];
  int nargs;
  int REDI;

  for(;;){
    printf("> ");
    if(fgets(line_in, MAX_LINE_IN, stdin) == NULL) exit(0);
    line_in[strlen(line_in) - 1] = '\0';

    REDI=0;
    char *token = strtok(line_in, " ");
    nargs = 0;
    args[nargs++] = token;
    while(token != NULL){
      token=strtok(NULL, " ");
      if(token != NULL){
        args[nargs++]=token;
        if(strcmp(token,">")==0){
          REDI=nargs;
        }
      }
    }
    args[nargs] = NULL;
  
    if(strcmp(args[0], "exit")==0) break;
    if(fork()==0){
      if(REDI!=0){
        redirect(args, REDI);
      }

     execvp(args[0],args);
      printf("command not found\n");
      exit(1);
    }
    else{
      wait(&status);
    }
  }
  return 0;
}

試したこと

『これにリダイレクトします。』というところは>を実行した時に表示されていたので、redirectの関数の中には入っているようです。またその関数の中に入ったargs[REDI]の文字列もレダイレクトの対象とするテキストファイル名が入っていたので問題ないと思います。

このコンパイルに関してはエラーはありませんでした。

実行
> echo hello > text.txt

これを実行した時は
text.txtのファイルに　hello > text.txt という文字列が入りました。

実行
> cal > text.txt

とした時には　cal: year `text.txt' not in range 1..9999
というエラーが出てtext.txtには何も入りませんでした。

補足情報（FW/ツールのバージョンなど）

dodox86

2018/06/03 07:07 編集

実行を試されている状況が良く分からないのですが、コンパイル(ビルド)したプログラムの実行ファイル名は何なのでしょうか。また、Windowsなのか、unix系(linux?)なのかを書いた方がより回答を得やすいと思います。（fork()を使われているのでunix系だとは思いますが、cygwinと言うことも考えられるので）

oika77

2018/06/03 08:12 編集

ご指摘ありがとうございます。macを使用しております。実行の部分では初めの>は無視してください。（コードの真ん中あたりのprintf("< ");から来ています。）

行動規範の内容に同意します

回答1件

ベストアンサー

redirect関数を呼び出すときの引数が不適切なのが原因です。

echo hello > text.txt

これを実行する際、echoへの引数は

args[0] = "echo"
args[1] = "hello"
args[2] = NULL

が期待ですがご質問のコードでは

args[2] = ">"
args[3] = "text.txt"
args[4] = NULL

と、リダイレクト指示がそのまま引数になってしまってます。これだとechoは
args[1]～args[3]までの３つのパラメータを印字しようとしますし、calは意味不明なパラメーター
">" "text.txt"を受け取ってしまいますのでエラーを引き起こします。

とりあえず動くようにするためには以下の(A)を追加すればよいです。

C
1void redirect(char *args[], int REDI){
2  int fd;
3
4  fd=open(args[REDI], O_WRONLY | O_CREAT, 0664);
5  close(STDOUT_FILENO);
6  dup(fd);
7
8 // ">"とその次のリダイレクト先のファイル名を起動コマンドへ渡さないようにする
9  args[REDI - 1] = NULL;  //(A)
10  execvp(args[0],args);
11}