diff options
author | Brad King <brad.king@kitware.com> | 2022-05-26 15:00:52 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2022-05-26 16:29:41 (GMT) |
commit | 0d201ad25ba4ddf83b4f17986ff5fd032c9f209e (patch) | |
tree | 49abe450b9e14d12de238562d6033398273d7b9b /Utilities | |
parent | 512c1f8f75d2536f7db5560fc48d4db8f539c931 (diff) | |
download | CMake-0d201ad25ba4ddf83b4f17986ff5fd032c9f209e.zip CMake-0d201ad25ba4ddf83b4f17986ff5fd032c9f209e.tar.gz CMake-0d201ad25ba4ddf83b4f17986ff5fd032c9f209e.tar.bz2 |
zlib: Add initialization to satisfy valgrind
Restore the change from commit cf133ff6b3 (Fix uninitialized variable
access in zlib, 2009-11-18, v2.8.2~705). Their web site claims it does
no harm ( https://www.zlib.net/zlib_faq.html#faq36), but fixing it this
way eliminates the diagnostic.
Diffstat (limited to 'Utilities')
-rw-r--r-- | Utilities/cmzlib/deflate.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/Utilities/cmzlib/deflate.c b/Utilities/cmzlib/deflate.c index 29ce1f6..5ec8374 100644 --- a/Utilities/cmzlib/deflate.c +++ b/Utilities/cmzlib/deflate.c @@ -285,6 +285,13 @@ int ZEXPORT deflateInit2_(strm, level, method, windowBits, memLevel, strategy, s->hash_shift = ((s->hash_bits+MIN_MATCH-1)/MIN_MATCH); s->window = (Bytef *) ZALLOC(strm, s->w_size, 2*sizeof(Byte)); + + /* The following memset eliminates the valgrind uninitialized warning + "swept under the carpet" here: + http://www.zlib.net/zlib_faq.html#faq36 */ + + memset(s->window, 0, s->w_size*2*sizeof(Byte)); + s->prev = (Posf *) ZALLOC(strm, s->w_size, sizeof(Pos)); s->head = (Posf *) ZALLOC(strm, s->hash_size, sizeof(Pos)); |