From 45e43e169527e7a98a8c8a821d37bf25822b764d Mon Sep 17 00:00:00 2001 From: Lasse Collin Date: Mon, 10 Mar 2008 13:41:25 +0200 Subject: [PATCH] Don't fill allocated memory with 0xFD when debugging is enabled. It hides errors from Valgrind. --- src/liblzma/common/allocator.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/liblzma/common/allocator.c b/src/liblzma/common/allocator.c index edea0f68..c5970312 100644 --- a/src/liblzma/common/allocator.c +++ b/src/liblzma/common/allocator.c @@ -36,9 +36,10 @@ lzma_alloc(size_t size, lzma_allocator *allocator) ptr = malloc(size); #if !defined(NDEBUG) && defined(HAVE_MEMSET) - // This helps to catch some stupid mistakes. - if (ptr != NULL) - memset(ptr, 0xFD, size); + // This helps to catch some stupid mistakes, but also hides them from + // Valgrind. Uncomment when useful. +// if (ptr != NULL) +// memset(ptr, 0xFD, size); #endif return ptr;