C言語についてです(マルチプロセス)

Question

前提・実現したいこと

簡単なshellのようなものを実装したいのですが、うまくいきません。

発生している問題・エラーメッセージ

segmentation fault

該当のソースコード

#include<stdio.h>
#include<unistd.h>  
#include<sys/wait.h>
#include<string.h>


int main(int argc,char argv[]){

        char command[100];
        printf(">");
        fgets(command,100,stdin);


        p=strtok(command," ");
        while(p!=NULL){
        p=strtok(NULL," ");
        }

        pid_t pid = fork();
    if (pid > 0 ){
   
        int ret;
        waitpid(pid,&ret,0);

    }
    else{
        char* cmd[]={p[0],p[1],NULL};
        execvp(cmd[0], cmd);
    }

}

試したこと

うまく文字列を空白で切り分けられているか、試したのですがsegmentation faultになっていました。

#include<stdio.h>
#include<unistd.h>
#include <string.h>

int main(){
        char command[200];
        char *p;
        printf(">");
        fgets(command,200,stdin);

        char *tp = strtok(command," ");
        while(tp!=NULL){
            tp=strtok(NULL, " ");
         
}
   printf("%s\n", tp);
        }

よろしくお願いいたします。

Comments

[yoorwm]

どの辺がマルチプロセスなのでしょうか？

Answer 1

strtok関数とはそもそも何なのかの理解が間違っているか、何なのかは理解しているが、使い方をちゃんと考えてないか、どちらかです。
strtokの返り値は何なのかを考察しましょう。

あと、cmd[]のあたりを見ると、文字、文字列、文字へのポインタのあたりの理解も出来ていませんので、文字列について復習しましょう。

Comments

[Cchan]

ありがとうございます

下記コードでは,表示はされるのですが、segmentationfaultになるのはなぜでしょうか・・・

```
#include<stdio.h>
#include<unistd.h>
#include <string.h>

int main(){
char command[1000];
char *p;
printf(">");
fgets(command,200,stdin);

char *tp = strtok(command," ");
printf("%s\n", tp);
while(tp!=NULL){
tp=strtok(NULL, " ");
printf("%s\n", tp);
}

}
```

[otn]

strtokの返り値は何なのかを考察しましょう。

Answer 2

以下のコードの原因について。

#include<stdio.h>
#include<unistd.h>
#include <string.h>

int main(){
	char command[1000];
	char *p;
	printf(">");
	fgets(command,200,stdin);

	char *tp = strtok(command," ");
	printf("%s\n", tp);
	while(tp!=NULL){
		tp=strtok(NULL, " ");
		printf("%s\n", tp);
	}
}

strtokの処理が間違っています。
strtokの戻り値はNULLポインターの可能性があるので、strtokの直後にprintfを行うとNULLポインターの場合エラーになります。
次のように書き換えればOK。

	char *tp = strtok(command," ");
	while(tp!=NULL){
		printf("%s\n", tp);
		tp=strtok(NULL, " ");
	}

Comments

[Cchan]

ありがとうございます！

Answer 3

質問のエラー原因は

printf("%s\n", tp);

```が実行される時点で tp が NULL になっている事が理由だと思います

Answer 4

while(tp!=NULL){

tp=strtok(NULL, " ");
}

これはなにをさせようとしてますか？
strtok が意味不明な上に、このループを抜けるとtpはNULLですが、それでいいんでしょうか

Comments

[Cchan]

ありがとうございます

下記コードでは,表示はされるのですが、segmentationfaultになるのはなぜでしょうか・・・

```
#include<stdio.h>
#include<unistd.h>
#include <string.h>

int main(){
char command[1000];
char *p;
printf(">");
fgets(command,200,stdin);

char *tp = strtok(command," ");
printf("%s\n", tp);
while(tp!=NULL){
tp=strtok(NULL, " ");
printf("%s\n", tp);
}

}
```