为什么释放动态分配的内存会在此处产生问题？

Question

我有以下代码：

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

char* creatString();
void printWordsThatStartWithLETTER(char letter, char *str);

void main()
{
    char *strEx1;
    char letter;
    strEx1=creatString();
    printf("Enter a letter:\n");
    scanf("%c",&letter);
    printWordsThatStartWithLETTER(letter, strEx1);
    free(strEx1);
}

char* creatString()
{
    char *str, *strTemp;
    int size;
    strTemp=(char*)malloc(256);
    printf("enter your string:\n");
    flushall();
    gets(strTemp);
    size = strlen(strTemp);
    str=(char*)malloc(size);
    strcpy(str,strTemp);
    //puts(str);
    free(strTemp);
    return str;
}
void printWordsThatStartWithLETTER(char letter, char *str)
{
    int sizeOfStrinf, i;
    sizeOfStrinf = strlen(str);
    for(i=0;i<sizeOfStrinf;i++)
    {
        if((str[i]==letter)||(str[i]==letter-32))
        {
            if(i==0)
            {
                while(str[i]!=32)
                {
                    printf("%c",str[i]);
                    i++;
                }
                printf("\n");
            }
            else
                if(str[i-1]==32)
                {
                    while(str[i]!=32)
                    {
                        printf("%c",str[i]);
                        i++;
                    }
                    printf("\n");
                }
        }
    }
}

它不会释放strEx1 ，我溢出了。 如何正确释放strEx1 ？

Answer 1

您在代码中实际遇到的问题

size = strlen(strTemp);
str=(char*)malloc(size);

基本上，您只有一个元素，没有空终止符。 所以稍后使用strcpy()

 strcpy(str,strTemp);

导致超出范围的访问，从而引发未定义的行为 。

你应该做类似的事情

 str = malloc(size + 1);

那就是

1. 请参见有关为什么不在C中malloc()和family的返回值的讨论。 。
2. 在使用返回值之前，请务必检查malloc()的返回值是否成功，以避免空指针取消引用。
3. 您一定不能使用gets() 。

Answer 2

您分配的内存太短了一个字节。 加一

size = strlen(strTemp);    
str=(char*)malloc(size+1);

或代替strcpy使用strncpy

size = strlen(strTemp);
str=(char*)malloc(size);
strncpy(str,strTemp,size);

strncpy函数将strSource的初始计数字符复制到strDest并返回strDest。 如果count小于或等于strSource的长度，则不会将空字符自动添加到复制的字符串中。 如果count大于strSource的长度，则使用空字符填充目标字符串，直到长度count。 如果源字符串和目标字符串重叠，则strncpy的行为未定义