c++ - cstring へのポインターを指定して strcpy() を使用すると、セグメンテーション違反が発生します

Question

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

    // checked for right number of arguments

    // reading pack name
    char* filename[MAX_STR_LEN];
    strcpy(*filename, argv[1]);  // seg fault here
}

Answer 1

char* filename[MAX_STR_LEN];

これはポインターの配列を宣言し、それらを何かを指すように初期化しません。それらを逆参照すると、未定義の動作が発生します。運が良ければ、セグメンテーション違反です。

おそらく、文字列を含む文字の配列を宣言するつもりでした:

char filename[MAX_STR_LEN];
strcpy(filename, argv[1]);

あとは、配列が十分な大きさであることを確認する必要があります。または、低レベルの C アーティファクトをいじるのをやめて、std::string代わりに使用します。

Answer 2

char filename[MAX_STR_LEN] // without the * 
strcpy(filename, argv[1]); // without the *

より良い（stackoverflowを避けるために！（笑））：

strncpy( filename, argv[1], sizeof(filename)-1 );
filename[ sizeof(filename)-1 ] = 0; 

Answer 3

1) filename は *char の配列です。それがあなたがやりたいことかどうかわかりません。あなたはおそらく持っていたい

char filename[FOO];

2) strcpy は (char *, const char *) を期待していますが、これを渡していますが、ここには割り当てられたスペースがありません。(1) を参照してください。