From 1b0cfd03d55fa60c1f5c0dce9f9d73f301ab7b8b Mon Sep 17 00:00:00 2001 From: "Ronald S. Bultje" Date: Thu, 16 Dec 2004 23:57:26 +0000 Subject: [PATCH] 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). --- ChangeLog | 6 ++++++ gst-libs/gst/riff/riff-read.c | 16 +++++++++------- 2 files changed, 15 insertions(+), 7 deletions(-) diff --git a/ChangeLog b/ChangeLog index dee092a8d9..d64511f177 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2004-12-17 Ronald S. Bultje + + * gst-libs/gst/riff/riff-read.c: + (gst_riff_read_strf_auds_with_data): + Read extradata correctly (fixes #155879). + 2004-12-16 David Schleef * gst/audioscale/gstaudioscale.c: allow passthru of >2 channel diff --git a/gst-libs/gst/riff/riff-read.c b/gst-libs/gst/riff/riff-read.c index d539471161..e77cb73edb 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); } }