[FFmpeg-cvslog] Init s->cdlms[][].recent to order - 1

Mashiat Sarker Shakkhar git at videolan.org
Wed Dec 7 04:57:32 CET 2011


ffmpeg | branch: master | Mashiat Sarker Shakkhar <shahriman_ams at yahoo.com> | Wed Nov 30 01:55:21 2011 +0600| [be8a0d26dbeec72b8e254e00724f170c28644c98] | committer: Mashiat Sarker Shakkhar

Init s->cdlms[][].recent to order - 1

Not sure if this is correct. But looks like
it was giving us off-by-one's when used to index
buffers.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=be8a0d26dbeec72b8e254e00724f170c28644c98
---

 libavcodec/wmalosslessdec.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/libavcodec/wmalosslessdec.c b/libavcodec/wmalosslessdec.c
index d4b245a..4be163e 100644
--- a/libavcodec/wmalosslessdec.c
+++ b/libavcodec/wmalosslessdec.c
@@ -776,7 +776,7 @@ static void reset_codec(WmallDecodeCtx *s)
     s->mclms_recent = s->mclms_order * s->num_channels;
     for (ich = 0; ich < s->num_channels; ich++) {
         for (ilms = 0; ilms < s->cdlms_ttl[ich]; ilms++)
-            s->cdlms[ich][ilms].recent = s->cdlms[ich][ilms].order;
+            s->cdlms[ich][ilms].recent = s->cdlms[ich][ilms].order - 1;
         /* first sample of a seekable subframe is considered as the starting of
            a transient area which is samples_per_frame samples long */
         s->channel[ich].transient_counter = s->samples_per_frame;



More information about the ffmpeg-cvslog mailing list