Cでの単純な文字列ランタイムエラー？

Question

このコードはうまくコンパイルされますが、実行中にセグメンテーションフォールトエラーが発生しますか？なぜ誰に言えますか？

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

int main() { 
    const char s2[] = "asdfasdf"; 
    char* s1; 

    strcpy(s1, s2); 
    printf("%s", s1); 

    return 0; 
} 

Answer 1

あなたは、単一のポインタ、s1のためのスペースが割り当てられますが、バイトがs1によって指しません。

s1 = (char *)malloc(strlen(s2) + 1); 
strcpy(s1, s2); 

あなたは文字の数よりも（mallocへの呼び出しで1）メモリの1つの以上のバイトを割り当てる必要があることに注意してください：

ソリューションは、動的にs1にメモリを割り当てることですs2最後に暗黙のNULLバイトがあるためです。

詳細については、C Memory Management (Stack Overflow)を参照してください。

Answer 2

s1にメモリを割り当てていません。あなたはs1へのポインタを持っていますが、strcpyがs2の値をコピーするために割り当てられたメモリはありません。

char *s1 = malloc(strlen(s2) + 1); 

strcpy(s1, s2); 

Answer 3

宛先を割り当てる必要があります（using namespace std;はCではなくC++で、残りのコードはCです）。

Answer 4

問題は、s1に関連付けられているメモリがないことです。 strcpyはmalloc()を呼び出しません。

次のいずれかを行うことができます：

char s1[10];

または

char *s1 = malloc(10);

Answer 5

あなたはS1に任意のメモリを割り当てられていません。これは何も指すことのないポインタです。

char* s1 = malloc(sizeof(s2)); 
strcpy(s1, s2); 
printf("%s", s1); 
free(s1); 

Answer 6

ポインタs1にメモリを割り当てる必要があります。あなたがそれをしなければ、それは未知のどこかを指しているでしょう、そして、こうしてセグメンテーション違反に到着します。正しいコードは次のとおりです。

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

int main() { 
    const char s2[] = "asdfasdf"; 
    char* s1 = malloc(21 * sizeof(s2[0])); 
    strcpy(s1,s2); 
    printf("%s",s1); 
    return 0; 
} 

Answer 7

s1のスペースを割り当てる必要があります。うまく動作するかを誰も投稿しています、しかし、あなたは既存の文字列のためのスペースを割り当てるための簡単な方法をしたいし、新しいポインタにそれをコピーした場合、このようstrdupの使用：

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

using namespace std; 

int main() { 
    const char s2[] = "asdfasdf"; 
    char* s1; 

    s1 = strdup(s2); 
    printf("%s", s1); 

    return 0; 
} 

誰かがのstrdup先に述べたように、それはそれを使う方法になります。ほとんどのシステムは、標準ライブラリに入っているので、それをサポートする必要があります。しかし、明らかにいくつかはしません。したがって、エラーが返された場合は既に述べた方法で自分自身を書いてください。既に述べた方法を使用してください;）

Answer 8

この問題に対処するためのstrdup（String Duplicate）の可能性はまだ指摘されていません。

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

using namespace std; 

int main() { 
    const char s2[] = "asdfasdf"; 
    char* s1; 

    s1 = strdup(s2); // Allocates memory, must be freed later. 
    printf("%s", s1); 

    free(s1);   // Allocated in strdup, 2 lines above 
    return 0; 
}