关于内存释放的问题；

bjkillall

因为毕业设计的需要，小弟写了一个字符串切割函数，如下，其中pos代表起始位置范围从0开始，size代表要切割的长度，具体问题请看下面代码的注释，此函数忽略了异常情况的判断；

1. char* My_SubStr(char* str, int pos, int size)
   
2. {
   
3.     char* temp=(char*)calloc(100,sizeof(char));
   
4.     assert(temp != NULL);
   
5. 
   
6.     strcpy(temp,str);
   
7. 
   
8.     char* recall = temp+pos;
   
9.     recall[size] = '\0';
   
10.    
    
11.     free(temp);  //在此处释放temp的空间了
    
12.     temp=NULL;
    
13. 
    
14.     return recall; //此处返回的指针是指向temp的，但是temp的空间已经释了，这样会不会有问题；
    
15. }

复制代码

主要的问题就是上面那些，我自己已经做过验证了，我这么写是能够正确返回结果的，但是我不是很清楚的是temp的空间我已经释放掉了，而返回recall依旧指向那里，这样会不会有问题，小弟坐等各位的答案；

hellioncu

当然有问题

Ray001

{:3_182:}结果保存在哪里了？

beyond_touch

毕业设计写成这样

1. request a buffer as argument
2. allocate a chunk of dynamic memory as buffer(no free)
...

liubingqian

当然有问题了，没有问题是侥幸（虽然每次都碰巧“正常”的可能性也很大）。

jerryz920

这个叫做定时炸弹，没准啥时候就炸了。

Ray001

void My_SubStr(char** str, int pos)
{
  *str = *str+pos;
}
gaosanwang 发表于 2010-11-10 17:30

    你这个不对，楼主后面还有个size参数。另外，楼主的函数不改变源串。

bjkillall

回复 5# liubingqian


    那我把 temp设成一个全局变量来使用好吗？虽然说这个小函数是用来截取字符串的，但是不能改变输入字符串的内容，所以我只能对其副本进行操作；

bjkillall

回复 8# Ray001


    那我把temp设成一个全局的变量来处理，这样设计好么？