#include "stdafx.h"
#include "malloc.h"
#include "stdlib.h"
char *substr(char *s, int startloc ,int len);

int main(int argc, char* argv[])
{
    int i,j;

    char str[20]="asdfghjkl",*p;
    p=str;
    scanf("%d%d",&i,&j);
   
    p=substr(str,i,j);
    puts(p);
    return 0;
}

char *substr(char *s, int startloc ,int len)
{
    char *p,*q;
    int i=0;
     
    p=(char *)malloc(len*sizeof(char));
    if(p==NULL)
    {printf("ERROR!!");
    exit (0);
    }
    q=p;
    for(i=startloc;i<len+startloc;i++)
    {
        *p=*(s+i);
        p++;
    }
    *p='\0';
    s=q;
    free(p);
return(s);
        
}
怎么处理char *substr(char *s, int startloc ,int len)函数的返回值啊，这里一直出问题！求解惑

这样做是不对的，被调函数的生命周期结束，里面的变量也随之消亡了。S返回后内存也就释放了，所以P那一块就会出现问题。

char *substr(char *s, int startloc ,int len)
{char *p=s+startloc;
   while(s<p)s++;
   *(s+len)=0;
return(s);
       
}

char *substr(char *s, int startloc ,int len)
{
    char *p,*q;
    int i=0;
     
    p=(char *)malloc(len*sizeof(char));
    if(p==NULL)
    {printf("ERROR!!");
    exit (0);
    }
    q=p;//注意这里！
    for(i=startloc;i<len+startloc;i++)
    {
        *p=*(s+i);
        p++;
    }
    *p='\0';
    s=q;//还有这里
    free(p);//最后你释放了p，由于q=p、s=q，所以你释放的是什么应该清楚了吧。
return(s);
        
}
最后补充一点：malloc申请的空间不会随着被调函数的生命周期结束而消亡，必须用free（）函数去释放。要不然以后数据结构的链表啊之类的就没法做了。

  q=p;//注意这里！
    for(i=startloc;i<len+startloc;i++)
    {
        *p=*(s+i);
        p++;
    }
    *p='\0';
    s=q;//还有这里
    free(p);//最后你释放了p，由于q=p、s=q，所以你释放的是什么应该清楚了吧。
return(s);
        
恩，在for语句中p++；使这个指针一直在变化，我让q保留了这个新存储空间的首地址的。但是还是不对，这个申请得到的地址释放掉结果就不能得出来，即使s已经等于该存储空间的首地址了，后来我把free语句去掉结果是对了，问题就在释放申请的空间这里，还望能给详细点解释啊，谢谢啊

    q=p;
    for(i=startloc;i<len+startloc;i++)
    {
        *p=*(s+i);
        p++;
    }
    for(i=0;i<len;i++)
        *(s+i)=*(q+i);
    *(s+i)='\0';
    free(q);
   
    return(s) ;
原来是这么回事， s=q这里还是把申请得到的空间地址赋给s，当释放q后，s中的地址就变成不确定地址了，应该把申请空间的数据赋给s，这样就不会造成p为野指针了