一、题目1
void GetMemory(char* p)
{
p = (char*)malloc(100);
}
void Test(void)
{
char* str = NULL;
GetMemory(str);
strcpy(str, "hello word");
printf(str);
}
代码存在的问题：
1、调用GetMemory函数的时候，str的传参为值传递，p是str的临时拷贝，所以在GetMemory函数内部动态开辟空间的地址存放在p中的时候，不会影响str的值。调用完GetMemory函数之后，str依然为NULL指针。这会使strcpy函数调用失败，原因是strcpy函数对NULL指针解引用。
2、GetMemory函数中malloc申请的空间没有释放，造成内存泄漏。
3、malloc的返回值最好判断一下，因为malloc有可能开辟内存空间失败返回空指针NULL(但在这个题里面这不是核心的缺陷)。
改法1：返回p的值(因为malloc在堆区申请的空间出了函数是不销毁的)；释放内存，再将指针置空。
char* GetMemory()
{
char* p = (char*)malloc(100);
return p;
}
void Test(void)
{
char* str = NULL;
str = GetMemory();
strcpy(str, "hello word");
printf(str);
free(str);
str = NULL;
}
改法2：传址调用，在调用GetMemory函数时通过指针改变GetMemory函数外部的str的值；释放内存，再将指针置空。
void GetMemory(char** p)
{
*p = (char*)malloc(100);
}
void Test(void)
{
char* str = NULL;
GetMemory(&str);
strcpy(str, "hello word");
printf(str);
free(str);
str = NULL;
}
二、题目2
char* GetMemory(void)
{
char p[] = "hello world";
return p;
}
void Test(void)
{
char* str = NULL;
str = GetMemory();
printf(str);
}
代码存在的问题：返回栈空间地址问题，GetMemory函数内部创建的数组是临时的，虽然将数组的起始地址返回给了str，但是数组的内存出了GetMemory函数之后就被回收了，而str依然保存着数组的起始地址，这时如果printf使用了str，就会造成非法访问，因为str是野指针。
有返回栈空间地址问题的代码：
int* f1(void)
{
int x = 10;
return (&x);//括号不加也行
}
如果接受了这个函数返回的地址，就会造成野指针问题。
三、题目3
int* f2(void)
{
int* ptr;
*ptr = 10;
return ptr;
}
代码存在的问题：没有对ptr进行初始化，ptr里的是随机值，是野指针，对ptr解引用，造成非法访问。
四、题目4
void GetMemory(char** p, int num)
{
*p = (char*)malloc(num);
}
void Test(void)
{
char* str = NULL;
GetMemory(&str, 100);
strcpy(str, "hello");
printf(str);
}
代码存在的问题：
1、没有对申请的内存空间进行释放，要用free进行内存释放，然后将指针置空。
2、malloc的返回值最好判断一下，因为malloc有可能开辟内存空间失败返回空指针NULL(但在这个题里面这不是核心的缺陷)。
修改：用free将内存空间释放，然后将指针置空。
void GetMemory(char** p, int num)
{
*p = (char*)malloc(num);
}
void Test(void)
{
char* str = NULL;
GetMemory(&str, 100);
strcpy(str, "hello");
printf(str);
free(str);
str = NULL;
}
五、题目5
void Test(void)
{
char* str = (char*)malloc(100);
strcpy(str, "hello");
free(str);
if (str != NULL)
{
strcpy(str, "world");
printf(str);
}
}
代码存在的问题：
1、用free将内存空间释放之后，没有将指针置空。这时str为非空指针，进入if语句，由于str指向的内存空间已经还给操作系统，所以str成为野指针，这时在strcpy中使用str，造成非法访问。
2、malloc的返回值最好判断一下，因为malloc有可能开辟内存空间失败返回空指针NULL(但在这个题里面这不是核心的缺陷)。
修改：在free(str);之后将指针置空。
void Test(void)
{
char* str = (char*)malloc(100);
strcpy(str, "hello");
free(str);
str = NULL;
if (str != NULL)
{
strcpy(str, "world");
printf(str);
}
}