[FFmpeg-devel] [PATCH 10/17] avcodec/webvttdec: add some memory checks
u at pkh.me
Sun Sep 21 19:31:13 CEST 2014
On Sun, Sep 21, 2014 at 07:19:43PM +0200, Nicolas George wrote:
> Le jour de la Récompense, an CCXXII, Clément Bœsch a écrit :
> > So I did that and pushed that part of the patchset, adjusted to use a
> > bprint helper.
> > Unfortunately, this raises a warning because av_bprint_is_complete()
> > doesn't take a const parameter. I was too lazy to actually fix the API,
> > but didn't want to cast to hide the warning. If you can send a patch to
> > fix the bprint API that would be really appreciated. Thanks :)
> Pushed the attached patch to my tree: Michael can pull if no objection.
Doesn't this need at least a minor bump?
> But you could have not needed it and make the whole set even simpler if you
> had added av_bprint_finalize() to the helper itself.
Not really; in SAMI at least the buffer is allocated only once, and
finalized only at the end. This behaviour could be adopted in other
text subtitles demuxers.
> Even better, "*got_sub = sub->num_rects > 0" could have been factored
This chunk will change later
> Nicolas George
> From c0394614bacb38a3a303e80b246a76f37a28b391 Mon Sep 17 00:00:00 2001
> From: Nicolas George <george at nsup.org>
> Date: Sun, 21 Sep 2014 19:03:33 +0200
> Subject: [PATCH] lavu/bprint: add const to av_bprint_is_complete() argument.
> Signed-off-by: Nicolas George <george at nsup.org>
> libavutil/bprint.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> diff --git a/libavutil/bprint.h b/libavutil/bprint.h
> index 839ec1e..d1682fc 100644
> --- a/libavutil/bprint.h
> +++ b/libavutil/bprint.h
> @@ -179,7 +179,7 @@ void av_bprint_clear(AVBPrint *buf);
> * It may have been truncated due to a memory allocation failure
> * or the size_max limit (compare size and size_max if necessary).
> -static inline int av_bprint_is_complete(AVBPrint *buf)
> +static inline int av_bprint_is_complete(const AVBPrint *buf)
> return buf->len < buf->size;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 473 bytes
Desc: not available
More information about the ffmpeg-devel