//编写一个函数，从标准输入读取一个字符串，把字符串复制到动态分配的
//内存中，并返回该字符串的拷贝。这个函数不应该对读入字符串长度作任何限制
#include<stdio.h>
#include<stdlib.h>
char *readstring(void);
int main(void)
{
    char *str = readstring();
    if(str == NULL)
        printf("error\n");
    else
        printf("%s\n",str);
    return 0;
}

char *readstring(void)
{
    char *array,*p;
    char *ptr;
    int length = 0;
    array = (char *) malloc (sizeof(char));
    gets(ptr);
    if (ptr == NULL || array == NULL)
        return NULL;
    while(*(ptr + length) != '\0')
        length++;
    array = (char *) realloc (array,(length + 1) * sizeof(char));
    if(array == NULL)
        return NULL;
    for(int i = 0; i < length; i++)
        array[i] = *(ptr + i);
    array[length] = '\0';
    p = array;
    free(array);   

    return p;
}

gets(ptr)的时候ptr是野指针，崩溃。

什么样的指针称为野指针呢

定义的时候ptr会指向一个随机的地址，这个地址是不可预知的，这样的指针就叫做野指针。
而

这时array经过malloc分配体空间后，其所指向的空间是可使用，是安全的。
ptr=array;后ptr所指向的地址也就是安全的可以使用的，这样gets(ptr)的时候就可以使用了。
不过你只给array分配了一个字节大小的空间，所以如果你输入两个字符以上，那么就会对未申请的空间进行操作，有风险，也是不允许的。所以你实际应该使用getc去获取字符，然后循环读取。

理解了，谢谢

但是下面这种方法却可以运行，为什么呢
char *readstring(void)
{
    char *array;
    char *ptr;
    int length = 1;
    array = (char *) malloc (sizeof(char));
    gets(ptr);
    if (ptr == NULL || array == NULL)
    {
        return NULL;
    }
    while (*ptr != '\0' )
    {
        length++;
        array  = (char *) realloc (array , length * sizeof(char));
        if (array == NULL)
            return NULL;
        array[length - 2] = *ptr;
        ptr++;
    }
    array[length - 1] = '\0';
   
    return array;
}

你一定对“可以运行”有什么误解。
我一直打一个比方，某人顺利贩毒多次都没被抓到，赚了好多钱，他敢说自己“可以运行”么？

它只是恰好可以出结果，没有崩溃，但用的方法并不是正确的，是这样吧

嗯，就是这个意思。