Package: zlib1g
Version: 1:1.2.3.5.dfsg-1~debug1
Severity: minor
File: /usr/lib/libz.so.1.2.3.5
We are experiencing some bugs in our project (pymvpa) so I went on
'valgrind triage'. Few issues come up due to libz, so I built 'experimental'
version with hope that some could have been resolved. Nevertheless here is one:
==15629== Conditional jump or move depends on uninitialised value(s)
==15629== at 0x976F917: inflateReset2 (inflate.c:157)
==15629== by 0x976FA6B: inflateInit2_ (inflate.c:193)
==15629== by 0x976B5EC: gz_head (gzread.c:132)
==15629== by 0x976C037: gz_make (gzread.c:305)
==15629== by 0x976C314: gzread (gzread.c:402)
==15629== by 0xF759671: znzread (in /usr/lib/libznz.so.1.1.0)
==15629== by 0xF5436E4: ??? (in /usr/lib/libniftiio.so.1.1.0)
==15629== by 0xF54DD87: nifti_image_read (in /usr/lib/libniftiio.so.1.1.0)
==15629== by 0xF30C51B: ??? (in
/usr/lib/python-support/python-nifti/python2.5/nifti/_clib.so)
==15629== by 0x48DB17: PyEval_EvalFrameEx (in /usr/bin/python2.5)
==15629== by 0x48EC8E: PyEval_EvalFrameEx (in /usr/bin/python2.5)
==15629== by 0x48F4E0: PyEval_EvalCodeEx (in /usr/bin/python2.5)
==15629== Uninitialised value was created by a heap allocation
==15629== at 0x4C221A7: malloc (vg_replace_malloc.c:195)
==15629== by 0x97771C7: zcalloc (zutil.c:306)
==15629== by 0x976FA30: inflateInit2_ (inflate.c:187)
==15629== by 0x976B5EC: gz_head (gzread.c:132)
==15629== by 0x976C037: gz_make (gzread.c:305)
==15629== by 0x976C314: gzread (gzread.c:402)
==15629== by 0xF759671: znzread (in /usr/lib/libznz.so.1.1.0)
==15629== by 0xF5436E4: ??? (in /usr/lib/libniftiio.so.1.1.0)
and if we look at the code of inflate.c:
130:int ZEXPORT inflateReset2(strm, windowBits)
131:z_streamp strm;
132:int windowBits;
133:{
...
139: state = (struct inflate_state FAR *)strm->state;
...
157: if (state->wbits != windowBits && state->window != Z_NULL) {
158: ZFREE(strm, state->window);
159: state->window = Z_NULL;
160: }
...
187: state = (struct inflate_state FAR *)
188: ZALLOC(strm, 1, sizeof(struct inflate_state));
189: if (state == Z_NULL) return Z_MEM_ERROR;
190: Tracev((stderr, "inflate: allocated\n"));
191: strm->state = (struct internal_state FAR *)state;
192: state->window = Z_NULL;
193: ret = inflateReset2(strm, windowBits);
so probably in this case the issue is non-existant (at least in this case)
since
state->window = Z_NULL;
so condition is never evaluated to True, but if that was the logic -- to allow
uninitialized wbits while window is Z_NULL, it would be better to swap the
order of conditions in 157 to pacify valgrind for such nerveous users as I am
;) and allow to catch true positives ;)
cheers
-- System Information:
Debian Release: squeeze/sid
APT prefers unstable
APT policy: (901, 'unstable'), (900, 'testing'), (300, 'experimental')
Architecture: amd64 (x86_64)
Kernel: Linux 2.6.31-1-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_US, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash
Versions of packages zlib1g depends on:
ii libc6 2.10.2-5 Embedded GNU C Library: Shared lib
zlib1g recommends no packages.
zlib1g suggests no packages.
-- no debconf information
--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
