diff options
| author | FlavioJS <FlavioJS@54d463be-8e91-2dee-dedb-b68131a5f0ec> | 2007-12-30 05:43:51 +0000 |
|---|---|---|
| committer | FlavioJS <FlavioJS@54d463be-8e91-2dee-dedb-b68131a5f0ec> | 2007-12-30 05:43:51 +0000 |
| commit | 553178680fbdaacbdcf5e8d8d4c2a657362839c2 (patch) | |
| tree | 9effd674f86d3818ab9276d38fa451db6ea8c8f9 | |
| parent | a2b44a8872f55a3afe16ad20682bbc4f6c685a87 (diff) | |
| download | hercules-553178680fbdaacbdcf5e8d8d4c2a657362839c2.tar.gz hercules-553178680fbdaacbdcf5e8d8d4c2a657362839c2.tar.bz2 hercules-553178680fbdaacbdcf5e8d8d4c2a657362839c2.tar.xz hercules-553178680fbdaacbdcf5e8d8d4c2a657362839c2.zip | |
* Made the memory manager set allocated memory to 0xCD and freed memory to 0xDD. The memory manager no longer 'hides' uses of freed memory.
git-svn-id: https://rathena.svn.sourceforge.net/svnroot/rathena/trunk@11994 54d463be-8e91-2dee-dedb-b68131a5f0ec
| -rw-r--r-- | Changelog-Trunk.txt | 5 | ||||
| -rw-r--r-- | src/common/malloc.c | 18 |
2 files changed, 22 insertions, 1 deletions
diff --git a/Changelog-Trunk.txt b/Changelog-Trunk.txt index ba9d1459c..1a41b5b97 100644 --- a/Changelog-Trunk.txt +++ b/Changelog-Trunk.txt @@ -3,8 +3,11 @@ Date Added AS OF SVN REV. 5091, WE ARE NOW USING TRUNK. ALL UNTESTED BUGFIXES/FEATURES GO INTO TRUNK. IF YOU HAVE A WORKING AND TESTED BUGFIX PUT IT INTO STABLE AS WELL AS TRUNK. +2007/12/30 + * Made the memory manager set allocated memory to 0xCD and freed memory + to 0xDD. The memory manager no longer 'hides' uses of freed memory. 2007/12/29 - + * Fixed two missing @LDFLAGS@ in src/plugins/Makefile.in. [FlavioJS] * Implemented THE official Steal skill equation and game mechanics (basically version from /stable plus a few tweaks) (see topic:116540) * Added missing bAddStealRate reference to doc/item_bonus.txt diff --git a/src/common/malloc.c b/src/common/malloc.c index a65ee7eed..70d6f89af 100644 --- a/src/common/malloc.c +++ b/src/common/malloc.c @@ -112,6 +112,8 @@ char* _bstrdup(const char *chr) #ifdef USE_MEMMGR +#define DEBUG_MEMMGR + /* USE_MEMMGR */ /* @@ -226,6 +228,10 @@ void* _mmalloc(size_t size, const char *file, int line, const char *func ) p->next = unit_head_large_first; } unit_head_large_first = p; +#ifdef DEBUG_MEMMGR + // set allocated data to 0xCD (clean memory) + memset(p + sizeof(struct unit_head_large) - sizeof(int), 0xCD, size); +#endif *(int*)((char*)p + sizeof(struct unit_head_large) - sizeof(int) + size) = 0xdeadbeaf; return (char *)p + sizeof(struct unit_head_large) - sizeof(int); } else { @@ -284,6 +290,10 @@ void* _mmalloc(size_t size, const char *file, int line, const char *func ) head->size = size; head->line = line; head->file = file; +#ifdef DEBUG_MEMMGR + // set allocated memory to 0xCD (clean memory) + memset(head + sizeof(struct unit_head) - sizeof(int), 0xCD, size); +#endif *(int*)((char*)head + sizeof(struct unit_head) - sizeof(int) + size) = 0xdeadbeaf; return (char *)head + sizeof(struct unit_head) - sizeof(int); } @@ -367,6 +377,10 @@ void _mfree(void *ptr, const char *file, int line, const char *func ) } head->block = NULL; memmgr_usage_bytes -= head->size; +#ifdef DEBUG_MEMMGR + // set freed memory to 0xDD (dead memory) + memset(ptr, 0xDD, size_hash - sizeof(struct unit_head_large) + sizeof(int) ); +#endif FREE(head_large,file,line,func); } else { ShowError("Memory manager: args of aFree is freed pointer %s:%d@%s\n", file, line, func); @@ -382,6 +396,10 @@ void _mfree(void *ptr, const char *file, int line, const char *func ) ShowError("Memory manager: args of aFree is overflowed pointer %s line %d\n", file, line); } else { head->block = NULL; +#ifdef DEBUG_MEMMGR + // set freed memory to 0xDD (dead memory) + memset(ptr, 0xDD, block->unit_size - sizeof(struct unit_head) + sizeof(int) ); +#endif memmgr_usage_bytes -= head->size; if(--block->unit_used == 0) { /* ブロックの解放 */ |