realloc() 无法重新分配内存

Question

我正在编写一个文件复制程序，在该程序中我遇到了关于 realloc() 的困难。 请看下面的 sn-p（我写它是为了理解 realloc() 的工作原理）：-

int main(){
    char *p =(char *) malloc ( 10 ),*t;
    p = "this is";
    if (strlen (p)==7)
    {
        t = realloc ( p,14);
        if ( t==NULL)
        {
            printf ("no no\n");
        }
    }
    printf("%p\n%p\n%d",p,t,strlen(p));
    free (p);
    free (t);
    return 1;
}

输出

no no      //because realloc () is unable to reallocate  the memory
00450357   //address of p
00000000   //address of t

那么为什么 realloc() 无法重新分配内存并将其（其地址）分配给t？

编辑 我在 Windows 中使用代码块。

Answer 1

您已经用静态字符串的地址覆盖了 malloc 返回的值。然后realloc接收静态字符串的地址作为参数重新分配。

char *p =(char *) malloc ( 10 ),*t;
p = "this is";
t = realloc ( p,14);

你可能想要的是：

char *p =(char *) malloc ( 10 ),*t;
strcpy(p, "this is");
t = realloc ( p,14);

Answer 2

分配给p之后你正在做

p = "this is";

覆盖malloc() 返回的值。由于p 指向的内存不可释放，realloc() 会失败。

复制字符串做

strcpy(p, "this is");

虽然检查p有足够的内存分配，否则使用

strncpy(p, "this is", length_of_p);

Answer 3

realloc 的手册页指出，除非realloc 的第一个参数是NULL，否则它一定是由先前对malloc、calloc 或realloc 的调用返回的。

t = realloc (p, 14);

上面的语句将p 传递给realloc，它指向字符串文字"this is"，它具有静态存储持续时间并且是只读的，尽管没有const 限定。因此，您的程序调用了未定义的行为。（由于段错误，它在我的机器上崩溃了。）

此外，当您重新分配 p 字符串字面量的第一个元素的地址时，您会失去对动态分配内存的处理，从而导致内存泄漏。

char *p =(char *) malloc ( 10 ),*t;
p = "this is";

您实际需要做的是使用strcpy 将字符串文字的内容复制到动态分配的缓冲区。

// include the required headers

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

// explicitly state void in the parameter list of main
int main(void) {
    // do not cast the result of malloc
    char *p = malloc(10 * sizeof *p);
    char *t;
    if(p == NULL) {
        printf("malloc failed\n");
        return 1;
    }
    // beware of buffer overrun by strcpy
    // consider using the safer strncpy if it suits
    p = strcpy(p, "this is");
    if(strlen(p) == 7) {
        // save the result of realloc in t
        // in case, realloc fails to allocate
        t = realloc(p, 14);
        if(t == NULL) {
            printf("realloc failed\n");
            // handle it
            // p points to old buffer
        }
        else {
            t = NULL;
            // p points to new buffer
        }
    }

    // after your done with p, free it
    free(p);

    return 0;
}