[FFmpeg-devel] [PATCH] make packet_size in AVFormatContext unsigned

Michael Niedermayer michaelni
Wed Jun 17 02:33:15 CEST 2009


On Tue, Jun 16, 2009 at 06:12:53PM -0400, Ronald S. Bultje wrote:
> Hi,
> 
> On Tue, Jun 16, 2009 at 5:39 PM, Michael Niedermayer <michaelni at gmx.at>wrote:
> 
> > On Tue, Jun 16, 2009 at 05:24:01PM -0400, Ronald S. Bultje wrote:
> > > It changes the behaviour of buggy code, that's exactly why I mentioned
> > it.
> > > Could you comment on the suggested ways of fixing this bug in my original
> > > email, or could you maybe elaborate on how you would like to see this bug
> > > fixed? Do you agree that bug is there?
> >
> > i dont really care how its fixed but it should be fixed not wierdly
> > disscussed how it changes through the type change
> 
> 
> But how? I wouldn't know what reasonable min/max packet sizes of mpeg
> streams are.

you described a bug, so please reject values that trigger that bug
if you can quote the spec about stricter limits thats fine as well of
course

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090617/dc70b068/attachment.pgp>



More information about the ffmpeg-devel mailing list