[FFmpeg-devel] libavcodec/als: remove check for predictor order of a block

Michael Niedermayer michael at niedermayer.cc
Sat Sep 23 18:15:32 EEST 2017


On Fri, Sep 22, 2017 at 01:54:27PM +0530, Umair Khan wrote:
> On Thu, Sep 21, 2017 at 6:30 PM, Paul B Mahol <onemda at gmail.com> wrote:
> > On 9/21/17, Umair Khan <omerjerk at gmail.com> wrote:
> >> Hi Ronald,
> >>
> >> On Mon, Sep 11, 2017 at 10:27 PM, Ronald S. Bultje <rsbultje at gmail.com>
> >> wrote:
> >>> Hi Umair,
> >>>
> >>> On Mon, Sep 11, 2017 at 4:06 AM, Umair Khan <omerjerk at gmail.com> wrote:
> >>>
> >>>> On Sun, Sep 10, 2017 at 10:30 PM, Paul B Mahol <onemda at gmail.com> wrote:
> >>>> > On 9/10/17, Umair Khan <omerjerk at gmail.com> wrote:
> >>>> >> On Sun, Sep 10, 2017 at 9:52 PM, Thilo Borgmann
> >>>> >> <thilo.borgmann at mail.de
> >>>> >
> >>>> >> wrote:
> >>>> >>>
> >>>> >>> Hi,
> >>>> >>>
> >>>> >>> > This patch fixes this bug - https://trac.ffmpeg.org/ticket/5297
> >>>> >>>
> >>>> >>> this is yet another iteration of
> >>>> >>>
> >>>> >>> http://ffmpeg.org/pipermail/ffmpeg-devel/2017-June/213005.html
> >>>> >>
> >>>> >> I see. I was keepnig reference codec as the source of truth.
> >>>> >>
> >>>> >> How do I reproduce this segfault? -
> >>>> >> http://ffmpeg.org/pipermail/ffmpeg-devel/2016-November/202530.html
> >>>> >> Is it one of the fate tests failing?
> >>>> >
> >>>> > No.
> >>>>
> >>>> In that case there's not much which can be done with this bug.
> >>>> That being said, I'm pretty sure we will anyways end up removing this
> >>>> check.
> >>>
> >>>
> >>> I think your question is "how do I get access to this sample", right? You
> >>> could ask Michael.
> >>
> >> Paul also tried to fix this bug, hence I'm assuming that he already
> >> asked Michael about the sample file. Anyways, it'd not be difficult to
> >> fix this bug if we can reproduce it.
> >
> > Thilo do not want to apply Michael fix because he do not understand his patch.
> 
> Which fix are you talking about?
> 
> I see that Michael added this check in the first place -
> http://git.videolan.org/?p=ffmpeg.git;a=commit;h=18f94df8af
> If someone can provide me with the file which is causing the segfault,
> maybe I can look into it.

file sent privatly to umair

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In fact, the RIAA has been known to suggest that students drop out
of college or go to community college in order to be able to afford
settlements. -- The RIAA
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170923/31a44944/attachment.sig>


More information about the ffmpeg-devel mailing list