[FFmpeg-devel] [PATCH] lavfi/deshake: fix deshake crash issue.

mypopy at gmail.com mypopy at gmail.com
Sat Sep 22 05:33:27 EEST 2018


On Sat, Sep 22, 2018 at 3:25 AM Michael Niedermayer
<michael at niedermayer.cc> wrote:
>
> On Thu, Sep 20, 2018 at 10:39:37AM +0800, mypopy at gmail.com wrote:
> > On Wed, Sep 19, 2018 at 7:07 PM Michael Niedermayer
> > <michael at niedermayer.cc> wrote:
> > >
> > > On Tue, Sep 18, 2018 at 09:37:29PM +0800, Jun Zhao wrote:
> > > > Fixes ticket #7441.
> > > >
> > > > Signed-off-by: Jun Zhao <mypopydev at gmail.com>
> > > > ---
> > > >  libavfilter/vf_deshake.c |   12 +++++++-----
> > > >  1 files changed, 7 insertions(+), 5 deletions(-)
> > >
> > > this doesnt look correct
> > >
> > > the blocks that will be used are not going out of array? so the contrast
> > > calculated from them also should not
> > >
> > Do you mean the correct way is changing the pos calculated from:
> > pos = (y - i) * stride + (x - j);  to  pos = (y + i) * stride + (x + j)  ?
>
> Iam not the author of vf_deshake, but whatever aligns the checked blocks
> correctly, should be the correct solution

Agree, so I submitted V2 patch for this issue, Thanks.

>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> Modern terrorism, a quick summary: Need oil, start war with country that
> has oil, kill hundread thousand in war. Let country fall into chaos,
> be surprised about raise of fundamantalists. Drop more bombs, kill more
> people, be surprised about them taking revenge and drop even more bombs
> and strip your own citizens of their rights and freedoms. to be continued
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


More information about the ffmpeg-devel mailing list