diff options
author | Ronald S. Bultje <rbultje@ronald.bitfreak.net> | 2004-12-16 23:57:26 +0000 |
---|---|---|
committer | Ronald S. Bultje <rbultje@ronald.bitfreak.net> | 2004-12-16 23:57:26 +0000 |
commit | 6c5841870cbef5dff2a422de85a9fb5bd25a38f5 (patch) | |
tree | e3284c8a5f34c4429b1e0d98afdf5bdad6f069c7 /gst-libs | |
parent | 83acacc0f67979312c2bbe61559ad14c8bfe2042 (diff) |
gst-libs/gst/riff/riff-read.c: Read extradata correctly (fixes #155879).
Original commit message from CVS:
* gst-libs/gst/riff/riff-read.c:
(gst_riff_read_strf_auds_with_data):
Read extradata correctly (fixes #155879).
Diffstat (limited to 'gst-libs')
-rw-r--r-- | gst-libs/gst/riff/riff-read.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/gst-libs/gst/riff/riff-read.c b/gst-libs/gst/riff/riff-read.c index d5394711..e77cb73e 100644 --- a/gst-libs/gst/riff/riff-read.c +++ b/gst-libs/gst/riff/riff-read.c @@ -664,16 +664,18 @@ gst_riff_read_strf_auds_with_data (GstRiffRead * riff, /* size checking */ *extradata = NULL; - if (strf->size > GST_BUFFER_SIZE (buf)) { - g_warning ("strf_auds header gave %d bytes data, only %d available", - strf->size, GST_BUFFER_SIZE (buf)); - strf->size = GST_BUFFER_SIZE (buf); - } else if (strf->size < GST_BUFFER_SIZE (buf)) { + if (GST_BUFFER_SIZE (buf) > sizeof (gst_riff_strf_auds) + 2) { gint len; - len = GST_BUFFER_SIZE (buf) - strf->size - 2; + len = GST_READ_UINT16_LE (&GST_BUFFER_DATA (buf)[16]); + if (len + 2 + sizeof (gst_riff_strf_auds) > GST_BUFFER_SIZE (buf)) { + GST_WARNING ("Extradata indicated %d bytes, but only %d available", + len, GST_BUFFER_SIZE (buf) - 2 - sizeof (gst_riff_strf_auds)); + len = GST_BUFFER_SIZE (buf) - 2 - sizeof (gst_riff_strf_auds); + } if (len > 0) { - *extradata = gst_buffer_create_sub (buf, strf->size + 2, len); + *extradata = gst_buffer_create_sub (buf, + sizeof (gst_riff_strf_auds) + 2, len); } } |