[FFmpeg-devel] [PATCH] Improve documentation for libavutil/base64.h
Sun Jan 25 22:37:07 CET 2009
On date Sunday 2009-01-25 18:10:05 +0100, Michael Niedermayer encoded:
> On Sun, Jan 25, 2009 at 02:04:52AM +0100, Stefano Sabatini wrote:
> > Can you confirm that the attached patch is correct? (At least I can
> > confirm that it works with the test program.)
> I can confirm that
> * it could lead to a sechole if you missed something and i suspect you did
> * its more complex
> * you did not list any advantage of this change
Use the minimal size data required. But I'll check better for the
> > /**
> > - * encodes base64
> > - * @param src data, not a string
> > - * @param buf output string
> > + * Encodes in base-64 the data in \p src and puts the resulting string
> > + * in \p dst.
> > + *
> > + * @param dst_len lenght of the \p dst string, it has to be at least
> > + * 4/3 * N, where N is the smaller multiple of 3 greater than or equal
> > + * to \p src_size.
> this is even worse, you AGAIN document the current implementation limit
> but now its much more complex and even exploitable
Honestly, I cannot see your point here.
We have a function which fails if the buffer where to write cannot
contain the data to write into it.
We know which is the size of this data, so we have two choices:
1) we can fail if the data size provided by the user is not big
enough. In this case the documentation should give some hints
regarding the size of the buffer to be provided, ideally this
should be the minimal required, and this value shouldn't depend on
the implementation, for this reason it looks safe to mention it in
2) we can write in the buffer for its whole size, avoiding to go
beyond its size. The problem with this is that the user isn't able
to detect the failure, that's not a big problem since he can
compute the required size by its own.
Problem with the actual implementation is that it requires an
(apparently, but maybe I'm wrong) arbitrary data size of len * 4 / 3 +
12, so an user providing a buffer with the minimal required size will
But I'll check better the code and the spec, in the case I didn't
considered some special case.
FFmpeg = Fantastic & Faboulous Magnificient Philosophical Empowered Governor
More information about the ffmpeg-devel