質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

Q&A

解決済

2回答

3244閲覧

自作シェルで入力したコマンドがexecv関数で動作しない

ijuya_yika

総合スコア50

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

0グッド

0クリップ

投稿2017/11/18 02:57

編集2017/11/18 19:15

###前提・実現したいこと
C言語で自作シェルを作成しているのですがプログラムがコマンド(ls, pwd等)に反応してくれません。

試しにlsを入力しexecv関数に入る前に引数を確認したのですが,token[0]/usr/bin/ls, token[1]NULLになっていたので引数に問題はないと思うのですが、では何がいけないのでしょうか。

###該当のソースコード

C

1#include <stdio.h> 2#include <unistd.h> 3#include <string.h> 4#include <sys/wait.h> 5#define S 1000 6 7int main(int argc, char *argv[]){ 8 char* token[S]; 9 char usrin[S]; 10 int pid, status, count; 11 12 13 printf(">>"); 14 fgets(usrin, sizeof(usrin), stdin); 15 usrin[strlen(usrin)-1] = '\0'; 16 17 /* 入力をトークンに分ける */ 18 token[0] = strtok(usrin, " "); 19 for(count=1; count<S; ++count){ 20 token[count] = strtok(NULL, " "); 21 if(token[count]==NULL) 22 break; 23 } 24 25 /* token[0]を/usr/bin/コマンド の形にする */ 26 char tmp[100] = ""; 27 strcat(tmp, "/usr/bin/"); 28 strcat(tmp, token[0]); 29 strcpy(token[0], tmp); 30 31 pid = fork(); 32 33 if(pid==0){ 34 /* この時点で入力がlsの場合 token[0]="/usr/bin/ls" token[1]=NULLを確認済 */ 35 execv(token[0], token); 36 } 37 else { 38 wait(&status); 39 40 } 41 42 return 0; 43}

###execvpに変更後のソースコード

C

1#include <stdio.h> 2#include <unistd.h> 3#include <string.h> 4#include <sys/wait.h> 5#define S 1000 6 7int main(int argc, char *argv[]){ 8 char* token[S]; 9 char usrin[S]; 10 int pid, status, count; 11 12 13 printf(">>"); 14 fgets(usrin, sizeof(usrin), stdin); 15 usrin[strlen(usrin)-1] = '\0'; 16 17 18 token[0] = strtok(usrin, " "); 19 for(count=1; count<S; ++count){ 20 token[count] = strtok(NULL, " "); 21 if(token[count]==NULL) 22 break; 23 } 24 25 26 pid = fork(); 27 28 if(pid==0){ /* token[0] => ls, token[1] => NULL */ 29 execvp(token[0], token); 30 } 31 else { 32 wait(&status); 33 34 } 35 36 return 0; 37} 38

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

/usr/bin/ls/usr/bin/pwdというものが存在しないからでは?execvの戻り値を確認すべきですね。

自分でPATHの中から探すが、execvpを使うか。

投稿2017/11/18 03:05

otn

総合スコア84421

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

ijuya_yika

2017/11/18 03:55

ご回答ありがとうございます。ご指摘どおり/usr/binにlsやpwd等のコマンドはありませんでした。 /usr/binはローカルにログインしたユーザが使うバイナリファイル、/binはどの実行レベルでも必要なバイナリファイルと違いがあるのですね...。 "/usr/bin/"を"/bin/"に変更することで無事実行出来ました。
otn

2017/11/18 07:28

execvpを使うのが良いと思いますよ。
ijuya_yika

2017/11/18 19:11

execvp便利ですね、execvの代わりに使わせて頂きます。
guest

0

strcpy(token[0], tmp);でポインタに初期より長い値を入れてかぶっているからではないでしょうか?

token[0]=tmp;

とされでみてどうでしょうか?

投稿2017/11/18 03:37

編集2017/11/18 04:04
A.Ichi

総合スコア4070

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問