[FFmpeg-cvslog] r23302 - trunk/libavformat/oggenc.c

[Reimar Döffinger]

On Mon, May 24, 2010 at 05:35:39PM -0700, Baptiste Coudurier wrote:
> >>    "Whole pages are taken in order
> >>    from multiple logical bitstreams multiplexed at the page level.  The
> >>    logical bitstreams are identified by a unique serial number in the
> >>    header of each page of the physical bitstream.  This unique serial
> >>    number is created randomly and does not have any connection to the
> >>    content or encoder of the logical bitstream it represents."
> >
> >0, 1, 2, 3 is a perfectly valid random sequence.  If the intent is to
> >disallow consecutive numbers, the spec should say so.  As is, this
> >code is nothing but obfuscation and abuse.  If you insist, use a
> >proper PRNG, but stop torturing av_get_random_seed.  This may be your
> >code, but not even you may misuse other functions.
> >
> 
> Well, I've created av_get_random_seed after michael's request.
> Michael suggested me to use get_random_seed to create the UMID for
> MXF and said it _was_ enough. I don't see why it wouldn't for OGG.

AFAIK the UMID is supposed to be unique "globally" if possible, and
there are somewhat good arguments for it, but adding this amount of
code that has several potential issues without any justification seems
silly to me.