リダイレクトを実装したい

Question

Cで簡単なシェルを実装したいのですが、リダイレクトが実現できません

#include<stdio.h>
#include<stdlib.h>
#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
#define BUF_SIZE 256

int nol(const char *s){
  int i;
  FILE *f=fopen(s,"r");
  char buf[BUF_SIZE];
  
  if(!f){
    printf("ファイル読み込みエラー\n");
    exit(0);}

  i=0;

  while((fgets(buf,BUF_SIZE,f)) != NULL){
    i++;
  }
  if(fclose(f))exit(0);

  return i;
}
void redirect(const char *args[],int a){//a:<のある位置
  int fd,i;
  
  args[a]=NULL;
  fd=open(args[a+1],O_WRONLY | O_CREAT,0664);//rwrwrのこと
  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 i;
   
   ////tokenize
   for(;;){
     printf("> ");
     if(fgets(line_in,MAX_LINE_IN,stdin)==NULL)exit(0);
     line_in[strlen(line_in)-1]='\0';
     
     char *token = strtok(line_in," ");
     nargs=0;
     args[nargs++]=token;
     while(token!=NULL){
       token = strtok(NULL," ");
       if(token!=NULL){
	 args[nargs++]=token;
       }
     }
     args[nargs] = NULL;
     
     if(args[0]!=NULL){
       if(!strcmp(args[0],"exit"))
	 {
	   exit(1);
	   continue;
	 }
       
       
       
       
       if(fork()==0){
	 
	 if(!strcmp(args[0],"nol"))
	   {
	     printf("%d\n",nol(args[1]));
	     exit(1);
	   }
	 for(i=0;i<nargs;i++)
	   {
	     if(!strcmp(args[i],">"))
	       {
		 redirect(args[0],i);
		 exit(1);
	       }
	   }
	 execvp(args[0],args);
	 printf("command not found\n");
	 exit(1);
       }
       
       else{
	 wait(&status);
       }  
       
     } 
     
   }  
   return 0;
 }

コンパイルすると以下の警告がでます
shell.c: In function ‘redirect’:
shell.c:38:18: warning: passing argument 2 of ‘execvp’ from incompatible pointer type [-Wincompatible-pointer-types]
execvp(args[0],args);
^
In file included from shell.c:3:0:
/usr/include/unistd.h:581:12: note: expected ‘char * const*’ but argument is of type ‘const char **’
extern int execvp (const char *__file, char *const __argv[
^
shell.c: In function ‘main’:
shell.c:87:13: warning: passing argument 1 of ‘redirect’ from incompatible pointer type [-Wincompatible-pointer-types]
redirect(args[0],i);
^
shell.c:31:6: note: expected ‘const char **’ but argument is of type ‘char *’
void redirect(const char *args[],int a){//a:<のある位�

実行後、echoコマンドで出力先を任意のファイルにリダイレクトしたいのですが新しいファイルが生成されません。

Comments

[y_waiwai]

そのコードで具体的にどういう動作をさせたいんでしょうか？

[HO_Pollyanna]

echo (任意の文字列) > ___.txt 　　と入力したときにテキストファイルに任意の文字列を出力させたいと考えています

[y_waiwai]

で、今のコードで、echo (任意の文字列) は実行できるんでしょうか

[HO_Pollyanna]

実行できません

Answer 1

真っすぐな回答でなくて申し訳ないのですが、system callの帰値を全部調べましょう。ドキュメントを読むという話ではなく、以下のようにコードを書くという意味です。

if((fd=open(...)) == -1 ) {
   // NULL になってしまった原因の分析
}

Comments

[HO_Pollyanna]

回答有り難うございます。参考になります。

Answer 2

void redirect(const char *args[], int a)

こちらの最初の引数は、文字列へのポインタ(ポインタのポインタ)
それに対し、呼び出し側は、
> redirect(args[0], i);

　

char *args[MAX_ARGS];

なので、args[0] は、最初の文字列です。
従って、示すものが違うと思われます。
単に、
redirect(args, i);
で良いかなと、思いますがどうでしょう。

const の有無に関する警告は、それだけでは、無視しても動く事が多い(だから、警告)。その陰にもっと大きな問題がある可能性ですね。

ちょっと、Linuxまで動かして確認できないので、問題点の指摘まで。
(久々に、fork()システムコールを見た。どっかのコピペでやってませんか?)

Comments

[HO_Pollyanna]

本当にありがとうございました。解決できました。

Answer 3

extern int execvp (const char *file, char *const argv[
なので*argv[]は書き換え可能で無ければいけません。

void redirect(char *const args[],int a)

にしましょう。

Comments

[HO_Pollyanna]

回答有り難う御座います
該当の箇所を修正したのですが
echo hello > hello.txt
と入力してもhello.txtが現れません...

コンパイル時の警告
shell.c: In function ‘main’:
shell.c:87:13: warning: passing argument 1 of ‘redirect’ from incompatible pointer type [-Wincompatible-pointer-types]
redirect(args[0],i);
^
shell.c:31:6: note: expected ‘char * const*’ but argument is of type ‘char *’
void redirect(char *const args[],int a){//a:<のある位�
^