On Wed, Feb 04, 2015 at 02:21:01PM +0000, Vittorio Giovara wrote:
> --- a/avplay.c
> +++ b/avplay.c
> @@ -2257,6 +2257,11 @@ static int decode_thread(void *arg)
>      global_video_state = is;
>  
>      ic = avformat_alloc_context();
> +    if (!ic) {
> +        av_log(NULL, AV_LOG_FATAL, "Could not allocate context.\n");

fprintf?  That function needs to make up its mind about what way to use
to print errors.

IMO this should be squashed with 2/4.  If we have one commit per unchecked
malloc we'll add (what feels like) thousands of commits before we're done.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to