diff options
| author | Clément Bœsch <u@pkh.me> | 2015-02-08 14:27:51 +0100 |
|---|---|---|
| committer | Clément Bœsch <u@pkh.me> | 2015-02-08 14:31:29 +0100 |
| commit | eb7efaa9244720c5f2051d76d76faeec864eca7a (patch) | |
| tree | 64453dc39bf55b513bcbef60a676353d0f9109a9 | |
| parent | 009a57b4792e1abb494954b1d7d3bc29fc7fb6b6 (diff) | |
| download | ffmpeg-eb7efaa9244720c5f2051d76d76faeec864eca7a.tar.gz | |
avfilter/dctdnoiz: fix slice_h computation
ceilf() can only work if the reminder of the division is not 0.
This fixes memory errors with for instance:
ffmpeg -f lavfi -i testsrc=s=800x500 -threads 3 -vf dctdnoiz -frames:v 1 -f null -
| -rw-r--r-- | libavfilter/vf_dctdnoiz.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavfilter/vf_dctdnoiz.c b/libavfilter/vf_dctdnoiz.c index 61438bb124..098a119aba 100644 --- a/libavfilter/vf_dctdnoiz.c +++ b/libavfilter/vf_dctdnoiz.c @@ -534,7 +534,7 @@ static int config_input(AVFilterLink *inlink) /* each slice will need to (pre & re)process the top and bottom block of * the previous one in in addition to its processing area. This is because * each pixel is averaged by all the surrounding blocks */ - slice_h = (int)ceilf(s->pr_height / s->nb_threads) + (s->bsize - 1) * 2; + slice_h = (int)ceilf(s->pr_height / (float)s->nb_threads) + (s->bsize - 1) * 2; for (i = 0; i < s->nb_threads; i++) { s->slices[i] = av_malloc_array(linesize, slice_h * sizeof(*s->slices[i])); if (!s->slices[i]) |