[FFmpeg-devel] [PATCH 1/2] checkasm: add sbrdsp tests

Matthieu Bouron matthieu.bouron at gmail.com
Mon Jul 3 15:32:28 EEST 2017


On Fri, Jun 30, 2017 at 05:16:37PM +0200, Matthieu Bouron wrote:
> On Fri, Jun 30, 2017 at 03:55:52PM +0200, Michael Niedermayer wrote:
> > On Thu, Jun 29, 2017 at 10:53:06PM -0300, James Almer wrote:
> > > On 6/29/2017 10:14 PM, Henrik Gramner wrote:
> > > > On Fri, Jun 30, 2017 at 1:58 AM, Michael Niedermayer
> > > > <michael at niedermayer.cc> wrote:
> > > >> Program received signal SIGSEGV, Segmentation fault.
> > > >> 0x0000000000684919 in ff_sbr_hf_gen_sse ()
> > > > 
> > > >>    0x0000000000684909 <ff_sbr_hf_gen_sse+25>:   sub    %r9,%r8
> > > > 
> > > >> => 0x0000000000684919 <ff_sbr_hf_gen_sse+41>:   movaps (%rsi,%r8,1),%xmm0
> > > > 
> > > >> r9             0xdeadbeef00000080       -2401053092612145024
> > > > 
> > > > Another case of a 32-bit int being used as part of a 64-bit operation.
> > > 
> > > I can't reproduce it on my ArchLinux x86_64 environment for some reason,
> > > but based on what you said i assume the attached patch should fix it.
> > 
> > no crash occurs here with this, so it seems fixed
> 
> Should i push the patchset or wait a little bit longer ?

Patchset applied.

-- 
Matthieu B.


More information about the ffmpeg-devel mailing list