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

mypopy at gmail.com mypopy at gmail.com
Sat Sep 22 05:30:06 EEST 2018


On Sat, Sep 22, 2018 at 6:52 AM Colin NG <colin_ng at hotmail.com> wrote:
>
> 1) The crash is caused by accessing un-allocated  memory area.
> 2) You can't compute the contrast outside of search window (by default, rx=ry=16).
>

Yes, so I give other fix in https://patchwork.ffmpeg.org/patch/10426/,
can you help to review or comment? Thanks.

> ________________________________
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> on behalf of Michael Niedermayer <michael at niedermayer.cc>
> Sent: September 21, 2018 3:24 PM
> To: FFmpeg development discussions and patches
> Subject: Re: [FFmpeg-devel] [PATCH] lavfi/deshake: fix deshake crash issue.
>
> 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
>
> [...]
> --
> 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