Re: [FFmpeg-devel] [PATCH] dvbsubdec: fix division by zero in compute_default_clut

2016-11-10 Thread Andreas Cadhalpun
On 10.11.2016 17:42, Michael Niedermayer wrote: > On Wed, Nov 09, 2016 at 08:52:54PM +0100, Andreas Cadhalpun wrote: >> dvbsubdec.c |2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> 10b117a15971daf75d61efe486b961d3cc59d480 >> 0001-dvbsubdec-fix-division-by-zero-in-compute_default_cl

Re: [FFmpeg-devel] [PATCH] dvbsubdec: fix division by zero in compute_default_clut

2016-11-10 Thread Michael Niedermayer
On Wed, Nov 09, 2016 at 08:52:54PM +0100, Andreas Cadhalpun wrote: > On 09.11.2016 02:27, Michael Niedermayer wrote: > > On Tue, Nov 08, 2016 at 10:36:01PM +0100, Andreas Cadhalpun wrote: > >> Since the loop goes down to i = 0 the number of iterations (variable > >> count) is the original i, not i

Re: [FFmpeg-devel] [PATCH] dvbsubdec: fix division by zero in compute_default_clut

2016-11-09 Thread Andreas Cadhalpun
On 09.11.2016 02:27, Michael Niedermayer wrote: > On Tue, Nov 08, 2016 at 10:36:01PM +0100, Andreas Cadhalpun wrote: >> Since the loop goes down to i = 0 the number of iterations (variable >> count) is the original i, not i - 1. >> In the case of i originally being 1, this miscalculation resulted i

Re: [FFmpeg-devel] [PATCH] dvbsubdec: fix division by zero in compute_default_clut

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 10:36:01PM +0100, Andreas Cadhalpun wrote: > Since the loop goes down to i = 0 the number of iterations (variable > count) is the original i, not i - 1. > In the case of i originally being 1, this miscalculation resulted in > a division by zero. > > This problem was introdu

[FFmpeg-devel] [PATCH] dvbsubdec: fix division by zero in compute_default_clut

2016-11-08 Thread Andreas Cadhalpun
Since the loop goes down to i = 0 the number of iterations (variable count) is the original i, not i - 1. In the case of i originally being 1, this miscalculation resulted in a division by zero. This problem was introduced in commit 4b90dcb8493552c17a811c8b1e6538dae4061f9d. Signed-off-by: Andreas