[FFmpeg-devel] [PATCH] lavf/flvdec: init AVPacket::pos to FLVTAG offset

[Suman Kancherla (సుమన్)]

Thanks for taking time to review my patch! I appreciate it very much!!

"Suman-" is my ID on github; Unless it is objectionable, I intend to use it
for code submissions.

Thanks!
Suman


On Wed, Oct 19, 2016 at 12:03 PM, Michael Niedermayer <
michael at niedermayer.cc> wrote:

> On Tue, Oct 18, 2016 at 03:13:23PM -0700, Suman- wrote:
> > Current code doesn't initialize AVPacket::pos. Made it point to FLVTAG
> so flv_read_packet can decode from pos
> > ---
> >  libavformat/flvdec.c                    |  1 +
> >  tests/ref/seek/acodec-adpcm-swf         | 46
> ++++++++++++++++-----------------
> >  tests/ref/seek/acodec-adpcm-swf-trellis | 46
> ++++++++++++++++-----------------
> >  tests/ref/seek/lavf-flv_fmt             | 36 +++++++++++++-------------
> >  tests/ref/seek/vsynth_lena-flashsv      | 40
> ++++++++++++++--------------
> >  tests/ref/seek/vsynth_lena-flv          | 40
> ++++++++++++++--------------
> >  6 files changed, 105 insertions(+), 104 deletions(-)
>
> Is it intended that the author of this patch is set to "Suman-" ?
>
> If its incomplete then please resubmit the patch with correctly set
> author
>
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> Many things microsoft did are stupid, but not doing something just because
> microsoft did it is even more stupid. If everything ms did were stupid they
> would be bankrupt already.
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>


-- 
Many thanks!
Suman