mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2024-11-18 07:47:17 +00:00
Code Issues 1 Projects Releases Wiki Activity

gst/id3demux/gstid3demux.c: Make sure we don't return GST_FLOW_OK with a NULL buffer in certain cases where a read be...

Browse source 
Original commit message from CVS:
* gst/id3demux/gstid3demux.c: (gst_id3demux_trim_buffer),
(gst_id3demux_read_range):
Make sure we don't return GST_FLOW_OK with a NULL buffer in
certain cases where a read beyond the end of the file is
requested. Fixes #345930.
* gst/apetag/gsttagdemux.c: (gst_tag_demux_trim_buffer),
(gst_tag_demux_read_range):
Fix same issue here as well.
This commit is contained in:
Tim-Philipp Müller 2006-06-29 11:11:50 +00:00
parent 1614e03cdb
commit 5aa89bdd33
3 changed files with 43 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
ChangeLog
View file

@ -1,3 +1,15 @@
2006-06-29 Tim-Philipp Müller <tim at centricular dot net>
* gst/id3demux/gstid3demux.c: (gst_id3demux_trim_buffer),
(gst_id3demux_read_range):
Make sure we don't return GST_FLOW_OK with a NULL buffer in
certain cases where a read beyond the end of the file is
requested. Fixes #345930.
* gst/apetag/gsttagdemux.c: (gst_tag_demux_trim_buffer),
(gst_tag_demux_read_range):
Fix same issue here as well.
2006-06-29 Zaheer Abbas Merali <zaheerabbas at merali dot org>
* sys/ximage/gstximagesrc.c: (gst_ximage_src_ximage_get):

24
gst/apetag/gsttagdemux.c
View file

@ -434,7 +434,7 @@ gst_tag_demux_trim_buffer (GstTagDemux * tagdemux, GstBuffer ** buf_ref)
no_out_buffer:
gst_buffer_unref (buf);
*buf_ref = NULL;
return TRUE;
return FALSE;
}
static void
@ -1158,26 +1158,32 @@ gst_tag_demux_read_range (GstTagDemux * demux,
if (!gst_tag_demux_get_upstream_size (demux))
return GST_FLOW_ERROR;
if (in_offset + length >= demux->priv->upstream_size - demux->priv->strip_end)
if (in_offset + length >= demux->priv->upstream_size - demux->priv->strip_end) {
if (in_offset + demux->priv->strip_end >= demux->priv->upstream_size)
return GST_FLOW_UNEXPECTED;
in_length = demux->priv->upstream_size - demux->priv->strip_end - in_offset;
else
} else {
in_length = length;
}
ret = gst_pad_pull_range (demux->priv->sinkpad, in_offset, in_length, buffer);
if (ret == GST_FLOW_OK && *buffer) {
if (!gst_tag_demux_trim_buffer (demux, buffer))
goto error;
goto read_beyond_end;
}
return ret;
error:
if (*buffer != NULL) {
gst_buffer_unref (buffer);
*buffer = NULL;
read_beyond_end:
{
GST_DEBUG_OBJECT (demux, "attempted read beyond end of file");
if (*buffer != NULL) {
gst_buffer_unref (buffer);
*buffer = NULL;
}
return GST_FLOW_UNEXPECTED;
}
return GST_FLOW_ERROR;
}
static GstFlowReturn

26
gst/id3demux/gstid3demux.c
View file

@ -384,7 +384,7 @@ gst_id3demux_trim_buffer (GstID3Demux * id3demux, GstBuffer ** buf_ref)
no_out_buffer:
gst_buffer_unref (buf);
*buf_ref = NULL;
return TRUE;
return FALSE;
}
static GstFlowReturn
@ -933,25 +933,31 @@ gst_id3demux_read_range (GstID3Demux * id3demux,
if (!id3demux_get_upstream_size (id3demux))
return GST_FLOW_ERROR;
if (in_offset + length >= id3demux->upstream_size - id3demux->strip_end)
if (in_offset + length >= id3demux->upstream_size - id3demux->strip_end) {
if (in_offset + id3demux->strip_end >= id3demux->upstream_size)
return GST_FLOW_UNEXPECTED;
in_length = id3demux->upstream_size - id3demux->strip_end - in_offset;
else
} else {
in_length = length;
}
ret = gst_pad_pull_range (id3demux->sinkpad, in_offset, in_length, buffer);
if (ret == GST_FLOW_OK && *buffer) {
if (!gst_id3demux_trim_buffer (id3demux, buffer))
goto error;
goto read_beyond_end;
}
return ret;
error:
if (*buffer != NULL) {
gst_buffer_unref (buffer);
*buffer = NULL;
read_beyond_end:
{
GST_DEBUG_OBJECT (id3demux, "attempted read beyond end of file");
if (*buffer != NULL) {
gst_buffer_unref (buffer);
*buffer = NULL;
}
return GST_FLOW_UNEXPECTED;
}
return GST_FLOW_ERROR;
}
static GstFlowReturn