Closed Bug 407860 Opened 15 years ago Closed 15 years ago

[valgrind] conditional depends on unitialized data in png writer

Categories

(Core Graveyard :: Image: Painting, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: sayrer, Unassigned)

References

()

Details

(Keywords: valgrind)

See URL.

==29183== Conditional jump or move depends on uninitialised value(s)
==29183==    at 0x49B6D12: longest_match (deflate.c:1143)
==29183==    by 0x49B7D92: deflate_slow (deflate.c:1595)
==29183==    by 0x49B75F1: MOZ_Z_deflate (deflate.c:790)
==29183==    by 0x7E8F77B: MOZ_PNG_write_finish_row (pngwutil.c:1996)
==29183==    by 0x7E8F8F4: MOZ_PNG_write_filtered_row (pngwutil.c:2873)
==29183==    by 0x7E8F93B: MOZ_PNG_write_find_filter (pngwutil.c:2806)
==29183==    by 0x7E8CB75: MOZ_PNG_write_row (pngwrite.c:918)
==29183==    by 0x7E72C03: nsPNGEncoder::AddImageFrame(unsigned char const*, unsigned, unsigned, unsigned, unsigned, unsigned, nsAString_internal const&) (nsPNGEncoder.cpp:267)
==29183==    by 0x7E722B2: nsPNGEncoder::InitFromData(unsigned char const*, unsigned, unsigned, unsigned, unsigned, unsigned, nsAString_internal const&) (nsPNGEncoder.cpp:99)
==29183==    by 0x794BA31: nsCanvasRenderingContext2D::GetInputStream(char const*, unsigned short const*, nsIInputStream**) (nsCanvasRenderingContext2D.cpp:775)
==29183==    by 0x79926E5: nsHTMLCanvasElement::ToDataURLImpl(nsAString_internal const&, nsAString_internal const&, nsAString_internal&) (nsHTMLCanvasElement.cpp:380)
==29183==    by 0x7992C84: nsHTMLCanvasElement::ToDataURL(nsAString_internal&) (nsHTMLCanvasElement.cpp:317)
http://www.zlib.net/zlib_faq.html#faq36 sounds relevant here, and http://archives.devshed.com/forums/compression-130/zlib-1-2-1-valgrind-warnings-314459.html says longest_match is intentionally using uninitialised values and will cause this warning when there is no actual problem.
nice, ok. INVALID.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → INVALID
Keywords: valgrind
Product: Core → Core Graveyard
Duplicate of this bug: 581181
This innocuous bug will be fixed by checkin of libz-1.2.5, bug #573137.
Depends on: 573137
You need to log in before you can comment on or make changes to this bug.