strcpyのsegmentation faulｔ

質問

118 char* argv[argc+1];
120
121 for(i = 0; i < argc ; i++)
122 {
123 char tmp[BUF];
124 while(buf[k]!=' ' & buf[k]!='\0')
125 {
126 tmp[j] = buf[k];
127 j++;
128 k++;
129 }
130 tmp[j]='\0';
131 j=0;
132 k++;
133 strcpy(argv[i],tmp);
134 　　　　//arg v[array]=tmp;
135 printf("temp = %s argv = %s\n",tmp,argv[array]);
136 }
137 argv[i+1]=tmp2;

133番の部分が問題です
最初に回る時にはprintfの結果が出ます
temp = test argv = test

でも2回目からはerrorになるます
argv[1]は　char*刑なのでstrcpyに何の問題もないと思いますが
どんな問題でerrorがでるのでしょうか？
これはlineを呼んで
int main(int argc, char* argv[])
のargv部分をようにするためのcodeです
errorの理由とか char* argv[]部分を簡単に作る方法を知りたいです
お願いします

回答No.3

trapezium · ベストアンサー

> 最初に回る時にはprintfの結果が出ます

これも偶然だと思うけど。そもそも

> char* argv[argc+1];

これが初期化されてない。実体がないアドレスに strcpy() してる。

> argv[1]は　char*刑なのでstrcpyに何の問題もないと思いますが
> どんな問題でerrorがでるのでしょうか？

型が当ってればいいってものじゃありません。メモリの確保はプログラマの責任ですよ？
buf がそのまま使えるなら切り分けたあと
argv[i] = buf + j;
とかでもいいかもしれないし、buf が一時的なメモリなのなら、malloc() してコピーしなければなりません。もちろん切り分けた文字列を格納可能なサイズで。使い終わったら free() する責任もあります。

回答No.2

Tacosan

おっと #1 で
strcpy(buf, "hello world";
に閉じるかっこをつけ忘れた.

まあここは本題じゃないですが.

回答No.1

Tacosan

例えば
#include <string.h>
#include <stdio.h>

int main()
{
char buf[2];
strcpy(buf, "hello world";
printf("%s\n", buf);
return 0;
}

というプログラムに問題がないと思いますか?