使用free()释放内存不起作用

时间:2017-01-16 14:29:58

标签: c pointers malloc free

这是一个小程序,它填充一些数组并在屏幕上打印其内容:

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

typedef struct{
    double **plist;
    int plistSize;
} ParticleList;

void sendPar(int *n, int np){
    // allocate memory for struct pl
    ParticleList pl;

    // allocate memory  for ParticleList np times
    pl.plist = malloc(sizeof(ParticleList) * np);

    // allocate memory for lists of size n[k]
    for(int k=0; k<np; k++){
        pl.plist[k] = malloc(sizeof(double) * n[k]);
    }

    // write some data to the list
    for(int k=0; k<np; k++){
        for(int l=0; l<n[k]; l++){
            pl.plist[k][l] = 100000*k+100*l;
        }
        pl.plistSize = n[k];
    }

    // print data to check
    for(int k=0; k<np; k++){
        printf("Listsize: %d\n", n[k]);
        for(int l=0; l<n[k]; l++){
            printf("Processor %d, Entry %d, Value %lf\n", k, l, pl.plist[k][l]);
        }
    }

    free(pl.plist);
}

int main(){
    int np = 3;

    int n[np];
    n[0] = 2;
    n[1] = 4;
    n[2] = 7;

    sendPar(n, np);
}

这是输出:

Listsize: 2
Processor 0, Entry 0, Value 0.000000
Processor 0, Entry 1, Value 100.000000
Listsize: 4
Processor 1, Entry 0, Value 100000.000000
Processor 1, Entry 1, Value 100100.000000
Processor 1, Entry 2, Value 100200.000000
Processor 1, Entry 3, Value 100300.000000
Listsize: 7
Processor 2, Entry 0, Value 200000.000000
Processor 2, Entry 1, Value 200100.000000
Processor 2, Entry 2, Value 200200.000000
Processor 2, Entry 3, Value 200300.000000
Processor 2, Entry 4, Value 200400.000000
Processor 2, Entry 5, Value 200500.000000

如果我现在想要释放内存,使用free(pl)不起作用。我也尝试了free(pl.plist),它确实有效。但是我还记得plistSize没有被释放的记忆。在这里释放记忆的正确方法是什么?

4 个答案:

答案 0 :(得分:2)

此内存分配

pl.plist = malloc(sizeof(ParticleList) * np);
                  ^^^^^^^^^^^^^^^^^^^

没有意义。我想你的意思是

pl.plist = malloc( sizeof( double * ) * np);
                  ^^^^^^^^^^^^^^^^

此循环中的最后一个语句

// write some data to the list
for(int k=0; k<np; k++){
    for(int l=0; l<n[k]; l++){
        pl.plist[k][l] = 100000*k+100*l;
    }
    pl.plistSize = n[k];
    ^^^^^^^^^^^^^^^^^^^^
}

也没有意义,因为在外循环的每次迭代中都会覆盖标量对象pl.plistSize

要释放已分配的内存,您可以编写

for(int k=0; k<np; k++){
    free( pl.plist[k] )
}

free( pl.plist );

答案 1 :(得分:0)

PI是一个静态声明的变量,它包含一个指向内存位置的指针。它是结构中需要释放的数组。

答案 2 :(得分:0)

我看到的问题是,你要分配每个free(),导致内存泄漏。

一旦你pl.plist - d pl.plist[k],就无法释放单个pl.plist[k],因此解除分配必须按照相反的顺序进行分配。

首先释放每个pl.plist,然后释放{{1}}本身。

答案 3 :(得分:0)

问题可能是您误解了mallocfree的工作原理。

变量pl及其成员plistSize未由您分配。它在编译代码时由编译器分配。你不应该试图释放它。

free只有malloc