mirror of
https://gitlab.freedesktop.org/gstreamer/gstreamer.git
synced 2025-02-17 03:35:21 +00:00
ext/jpeg/gstjpegdec.*: Fix crashes/invalid memory access for pictures that have a height that is not a multiple of 16...
Original commit message from CVS: * ext/jpeg/gstjpegdec.c: (gst_jpeg_dec_init), (gst_jpeg_dec_chain), (gst_jpeg_dec_change_state): * ext/jpeg/gstjpegdec.h: Fix crashes/invalid memory access for pictures that have a height that is not a multiple of 16 (or rather: v_samp_factor * DCTSIZE). Also fix the state change function for downwards state changes (need to chain up to parent before destroying our resources, to make sure pads get deactivated and our chain function isn't running and using those very same resources in another thread). The jpeg line buffer only needs to be v_samp_factor*DCTSIZE lines per plane, not picture_height lines; allocate that on the stack.
This commit is contained in:
parent
9b36e6d56e
commit
0e58d0f804
1 changed files with 16 additions and 0 deletions
16
ChangeLog
16
ChangeLog
|
@ -1,3 +1,19 @@
|
|||
2005-08-11 Tim-Philipp Müller <tim at centricular dot net>
|
||||
|
||||
* ext/jpeg/gstjpegdec.c: (gst_jpeg_dec_init), (gst_jpeg_dec_chain),
|
||||
(gst_jpeg_dec_change_state):
|
||||
* ext/jpeg/gstjpegdec.h:
|
||||
Fix crashes/invalid memory access for pictures that have a height
|
||||
that is not a multiple of 16 (or rather: v_samp_factor * DCTSIZE).
|
||||
|
||||
Also fix the state change function for downwards state changes
|
||||
(need to chain up to parent before destroying our resources, to
|
||||
make sure pads get deactivated and our chain function isn't
|
||||
running and using those very same resources in another thread).
|
||||
|
||||
The jpeg line buffer only needs to be v_samp_factor*DCTSIZE lines
|
||||
per plane, not picture_height lines; allocate that on the stack.
|
||||
|
||||
2005-08-10 Tim-Philipp Müller <tim at centricular dot net>
|
||||
|
||||
* gst/wavparse/gstwavparse.c: (gst_wavparse_stream_headers),
|
||||
|
|
Loading…
Reference in a new issue