On Wed, Jan 07, 2015 at 10:31:41AM -0800, Mark Reid wrote:
[...]
> +    segment = av_mallocz(sizeof(MXFIndexTableSegment));

sizeof(*segment)?

> +    if (!segment) {

> +        av_log(mxf->fc, AV_LOG_ERROR, "failed to allocate 
> IndexTableSegment");

small note: missing \n

You can probably drop the log altogether thought, this is unlikely to
happen anyway. 

> +        return AVERROR(ENOMEM);
> +    }
> +
[...]

-- 
Clément B.

Attachment: pgptTgyHuyUMB.pgp
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to