[FFmpeg-devel] [PATCH v2] libavformat/mxfdec.c: support demuxing opatom audio without index

[Clément Bœsch]

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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150108/1be2d7f4/attachment.asc>