인프런 커뮤니티 질문&답변

[강의 화면 사이드에서 이 질문을 보시는 분들은 [여기 (클릭하면 현재 강의 화면 페이지에서 이동하므로 주의)]서 보시면 따배씨 질문 페이지에서 조금 더 큰 화면으로 보실 수 있습니다.]

안녕하세요. 해당 강의에서 연습 문제를 직접 만들어보던 중에 getintfo()함수에서 runtime error가 발생하여 질문 드립니다.

 scanf() code line이 malloc()보다 먼저 실행되어야, 동적 할당 메모리를 해제할 때, runtime error가 발생하지 않는데 왜 그런 것인지 궁금합니다.

 

우선 제가 처음에 만들고나서 runtime error 발생 지점을 확인하기 위해 만든 code는 아래와 같습니다.

getinfo() 함수 부분을 보시면, malloc()으로 동적 메모리 할당을 먼저 받은 다음에, scanf()함수로 char buffer[SLEN]에 입력을 받도록 했었습니다.

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>
#include <stdlib.h>	// malloc(), free()
#include <string.h>	// strlen(), strcpy()

#define SLEN 81

struct namect {
	char* fname;	// Use malloc()
	char* lname;	// Use malloc()
	int letters;
};

void getinfo(struct namect*);		// Allocate memory
void makeinfo(struct namect*);
void showinfo(const struct namect*);
void cleanup(struct namect*);		// Free memory when it is done

int main()
{
	struct namect person;

	getinfo(&person);
	makeinfo(&person);
	showinfo(&person);
	cleanup(&person);

	return 0;
}

void getinfo(struct namect* ptr_nct)
{
	char buffer[SLEN] = { 0, };
	int flag = 0;

	ptr_nct->fname = (char*)malloc(strlen(buffer) + 1);	// Dynamic memory allocation for pointer 'fname'
	if (ptr_nct->fname != NULL)
	{
		printf("\nDynamic memory allocation for 'fname' is completed.\n");

		printf("Please enter your first name.\n>> ");
		flag = scanf("%[^\n]%*c", buffer);				// Taking string input into 'buffer'
		strcpy(ptr_nct->fname, buffer);					// Copy of string by pointer from 'buffer' to 'fname`
	}
	else
		printf("\nDynamic memory allocation for 'fname' is NOT completed.\n");


	ptr_nct->lname = (char*)malloc(strlen(buffer) + 1);	// Dynamic memory allocation for pointer 'lname'
	if (ptr_nct->lname != NULL)
	{
		printf("\nDynamic memory allocation for 'lname' is completed.\n");

		printf("Please enter your last name.\n>> ");
		flag = scanf("%[^\n]%*c", buffer);				// Taking string input into 'buffer'
		strcpy(ptr_nct->lname, buffer);					// Copy of string by pointer from 'buffer' to 'lname`
	}
	else
		printf("\nDynamic memory allocation for 'lname' is NOT completed.\n");
}

void makeinfo(struct namect* ptr_nct)
{
	ptr_nct->letters = strlen(ptr_nct->fname) + strlen(ptr_nct->lname);
}

void showinfo(const struct namect* ptr_nct)
{
	printf("%s %s, your name contains %d letters.\n", ptr_nct->fname, ptr_nct->lname, ptr_nct->letters);
}

void cleanup(struct namect* ptr_nct)
{
	// Deallocation of dynamically allocated memories
	printf("\nTime to free the allocated memory for 'fname'\n");
	free(ptr_nct->fname);	// Runtime error occurs in Visual Studio
	printf("Deallocation for 'fname' is completed!\n");

	printf("\nTime to free the allocated memory for 'lname'\n");
	free(ptr_nct->lname);	// Runtime error occurs in Visual Studio
	printf("Deallocation for 'lname' is completed!\n");
}

 

 

실행해보니, Heap Corruption Detected라는 error가 발생했습니다.

우선 어떤 code line에서 error가 발생한 것인지 확인해보기 위해, debugger로 line마다 확인해봤습니다.

---

cleanup()함수의 free()를 실행할 때, error가 발생하는 것을 확인했습니다.

그렇다면, 동적 메모리 할당은 정상적으로 수행되었는데, 해제를 할 때 문제가 발생한 것이라고 판단했습니다.

Google에 해당 error를 검색해보면, 이 Error가 발생하는 것에는 아래의 이유가 있다고 합니다.

• 메모리 할당이 되지 않은 포인터(NULL Pointer)에 값을 넣음
• 할당치를 초과하여 데이터가 입력됨

그런데, 제 경우 분명히 정상적으로 동적 할당을 먼저 받았으므로, fname과 lname을 입력할 수 있었습니다.

혹시라도 동적할당이 되지 않았는지, 체크하기 위해 동적 할당이 완료되었다는 메세지를 출력하도록 했습니다. 그리고 제가 입력한 string은 buffer + 1의 size인 82 bytes를 초과하지도 않았습니다.

도대체 왜 메모리 해제에서 문제가 발생하는지 고민하다가,

다른 compiler에서도 작동하는지 확인해보기 위해 GCC Compiler를 사용하도록 설정한 VS Code와, Online C Compiler에서 시도해봤습니다.

---

VS Code에서는 동적 할당 메모리가 해제되면서 프로그램이 정상적으로 종료됩니다.

---

Online C Compiler에서도 프로그램이 정상적으로 작동합니다.

---

하지만 Visual Studio에서 실행해보면, free()에서 더 이상 진행되지 않고, error가 발생한 것을 재차 확인했습니다.

 

그러다가buffer에 입력을 받는 scanf()의 code line이 malloc()보다 먼저 실행되도록 수정해봤습니다.

 

 

정상적으로 작동하는 것을 확인했습니다.

 

Q. 왜 scanf() code line이 malloc()보다 먼저 실행되어야, 동적 할당 메모리를 해제할 때, 문제가 없는 것인가요??