[FFmpeg-devel] fate: Do not report side data size

wm4 nfxjfg at googlemail.com
Thu Mar 9 08:48:53 EET 2017


On Thu, 9 Mar 2017 02:20:03 +0100
Michael Niedermayer <michael at niedermayer.cc> wrote:

> On Wed, Mar 08, 2017 at 11:54:59PM +0100, Hendrik Leppkes wrote:
> > On Wed, Mar 8, 2017 at 3:42 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:  
> > > Hi,
> > >
> > > On Wed, Mar 8, 2017 at 9:31 AM, wm4 <nfxjfg at googlemail.com> wrote:
> > >  
> > >> On Wed, 8 Mar 2017 14:09:53 +0100
> > >> Michael Niedermayer <michael at niedermayer.cc> wrote:
> > >>
> > >> If the size printing is removed then other code should be added  
> > >> > to test for the size to match the correct value  
> > >>
> > >> Then it would be more reasonable to make av_packet_add_side_data()
> > >> check whether the size is correct for the given side data type.  
> > >
> > >
> > > I think you're both right here, this is a pretty good idea (for fixed-size
> > > side-data types).
> > >  
> > 
> > So how do we fix fate now? Change the datatypes to uint32_t, remove
> > the size print out?  
> 
> > Shouldn't keep all 32-bit fate clients broken for much longer.  
> 
> +1
> 

You're the one stopping the simple fix.


More information about the ffmpeg-devel mailing list