搞不清到底有没有free掉

太平绅士

原帖由 cugb_cat 于 2009-3-3 15:29 发表
free(p);
p = NULL;

超鄙视这种写法

cugb_cat

原帖由 太平绅士 于 2009-3-3 15:33 发表


超鄙视这种写法

呵呵，鄙视就别这么写

wysunxiaohua

谢谢
原本已经加了p = NULL; 估计是程序bug

312-pirl0

应该这样作，
void *p;
p = NULL;

if (p !＝ NULL)
{
    free(p);
}

cobras

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

struct s_memory_block {
    unsigned long magid;
#define MEMORY_BLOCK_MAGID   0x010255AAUL
    size_t size;
};

void *memory_alloc(size_t size)
{
    struct s_memory_block *memblk;

    if (size > 0) {
        memblk = (struct s_memory_block *)malloc(size + sizeof(struct s_memory_block));
        if (memblk != NULL) {
            memset(memblk, 0, size + sizeof(struct s_memory_block));
            memblk->magid = MEMORY_BLOCK_MAGID;
            memblk->size = size;
            return memblk + 1;
        }
    }
   return NULL;
}

int memory_free(void *block)
{
    struct s_memory_block *memblk;

    if (block != NULL) {
        memblk = (struct s_memory_blk *)block - 1;
        if (memblk->magid == MEMORY_BLOCK_MAGID && memblk->size > 0) {
            memset(memblk, 0, memblk->size + sizeof(struct s_memory_block));
            free(memblk);
            return 0;
        }
    }
    return -1;
}

int memory_freed(void *block)
{
    struct s_memory_block *memblk;

    if (block != NULL) {
        memblk = (struct s_memory_blk *)block - 1;
        if (memblk->magid == MEMORY_BLOCK_MAGID && memblk->size > 0) {
            return 0;
        }
    }
    return -1;
}

然后用宏定义代替malloc和free

samon_fu

如果怕出错就自己包下，没必要搞太复杂：
[code]
void Free( void * ptr) {
    if (ptr != NULL) {
        free(ptr);
        ptr = NULL;
    }
}

不过其实很没必要，如果做free的人认为有必要做这些，早就放库里了。

free的接口人就是要求使用free的人自己按规则出牌。malloc与free成对使用。

samon_fu

代码是自己写的，free没free自己都不知道，那别人就更不知道了。

system888net

原帖由 cugb_cat 于 2009-3-3 15:29 发表
free(p);
p = NULL;

良好的工程学方法和习惯.

同时也不排斥其他的方法和习惯(前提是能有效规避"人"的习惯性风险,就如开车一样,道理大家都懂,但车祸照样天天有).

cugb_cat

原帖由 samon_fu 于 2009-3-3 20:14 发表
如果怕出错就自己包下，没必要搞太复杂：

void Free( void * ptr) {
    if (ptr != NULL) {
        free(ptr);
        ptr = NULL;
    }
}

不过其实很没必要，如果做free的人认为有必要做这些， ...

这样不行，改成宏吧