Re: [FFmpeg-devel] [PATCH 3/8] ffmdec: make sure the time base is valid

2015-03-09 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 12:34:54PM +0100, Andreas Cadhalpun wrote: > On 09.03.2015 03:59, Michael Niedermayer wrote: > >is anything using a 0/n timebase ? > > I don't think so. > > >if not i would extend this to also disallow 0/n > > OK, new patch attached. > > Best regards, > Andreas > > ff

Re: [FFmpeg-devel] [PATCH 3/8] ffmdec: make sure the time base is valid

2015-03-09 Thread Andreas Cadhalpun
On 09.03.2015 03:59, Michael Niedermayer wrote: is anything using a 0/n timebase ? I don't think so. if not i would extend this to also disallow 0/n OK, new patch attached. Best regards, Andreas >From be6482e2ecf8ea109e9d18f578dc5a17085dde43 Mon Sep 17 00:00:00 2001 From: Andreas Cadhalpu

Re: [FFmpeg-devel] [PATCH 3/8] ffmdec: make sure the time base is valid

2015-03-08 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 12:03:33AM +0100, Andreas Cadhalpun wrote: > Hi, > > attached patch fixes assertions in av_rescale_rnd. > > Best regards, > Andreas > ffmdec.c | 11 +++ > 1 file changed, 11 insertions(+) > 26bb6e42cc4cb5afc9ac83c27152edece8d62943 > 0003-ffmdec-make-sure-the-

[FFmpeg-devel] [PATCH 3/8] ffmdec: make sure the time base is valid

2015-03-08 Thread Andreas Cadhalpun
Hi, attached patch fixes assertions in av_rescale_rnd. Best regards, Andreas >From 8f647fe16693509056483c4d1bc4a57895e9de10 Mon Sep 17 00:00:00 2001 From: Andreas Cadhalpun Date: Sun, 8 Mar 2015 23:12:59 +0100 Subject: [PATCH 3/8] ffmdec: make sure the time base is valid A negative time base c