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) 메모리를 하나 더 바이트를 할당 할 필요가 유의 사항 :

해결책은 동적 s1 메모리를 할당하는 것입니다 s2 끝에 암시적인 NULL 바이트가 있기 때문입니다.

자세한 내용은 C Memory Management (Stack Overflow)을 참조하십시오.

Answer 2

s1에 메모리를 할당하지 않았습니다. 당신은 s1에 대한 포인터를 가지고 있지만 s2의 값을 복사하기 위해 strcpy에 할당 된 메모리는 없다.

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; 
}