一部コマンドでエラー

質問

以下のプログラムにパス検索機能を追加しろ。
/bin/lsと入力していたものを、lsだけで実行できるようにする。
そのほかにもdata -Iやgcc -o test test.c、./testを実行可能にする。

と言う課題で、 if(execv(argv[0],argv[])==(-1)){をif(execvp(argv[0],argv)==(-1)){とする方法を聞いたのですが、
gcc -o test test.cを入力すると長文のエラーが出力されます。

どうしたら良いでしょうか？

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
main(int argc, char *argv[]){
int len, pid, st;
char *p;
int i, sp_flag;

static char prompt[64]="> ";
char command[256];
printf("%s",prompt);

while (fgets(command, 256, stdin) != NULL){
if((len = strlen(command)) == 1)break;
command[len-1] = '\0';

p=(char *)&command;
sp_flag=0;

for(i=0; *p!='\0';p++){
if(sp_flag == 0 && *p!=' '){
argv[i]=p; sp_flag=1;
i++;
}else if(*p==' '){
*p='\0'; sp_flag=0;
}
}
argv[i]=(char *)0;
for(i=0; argv[i]!=(char *)0; i++){
printf("%s\n",argv[i]);
}

if((pid = fork()) == 0){
if(execv(argv[0],argv)==(-1)){
printf("%s", "exec error.\n"); exit(1);}
} else if(pid >= 1){
wait(&st);
printf("%s", prompt);
} else {
perror("fork");
exit(1);
}
}
exit(0);
}

回答No.3

Tacosan

argv だけことさら特別視する必要はないと思いますが＞#2.
確かに argv は (外部との関係で) 特殊な面はあるけど, とりあえずここではもっと単純に「配列で確保されている領域を超えて使っちゃダメ」でいいのでは?
あと, プログラムという面では
・トークンに切り分けるときの for が微妙. 間違ってはいないけど
for (i = 0, p = command; *p != '\0'; p++)
のように「p の初期化」も入れたほうが見る人に安心感を与えると思う
・fgets の引数 256 の根拠はなんだ
とかの突っ込みポイントはあります.

回答No.2

SilverThaw

>長文のエラーが出力されます。
よりも前にやることが一つ。

argv[]で確保されている領域は不定長なので、argv[]に対して値を入れるような処理は作らない。
途中の処理で明らかにメモリ破壊を発生する箇所がある。

回答No.1

notnot

エラーなど出ませんが？

どんなエラーが出ましたか？なぜ、エラーメッセージを書かなかったのですか？