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

Mark Reid mindmark at gmail.com
Sat Jan 10 03:08:04 CET 2015


On Thu, Jan 8, 2015 at 12:40 PM, Clément Bœsch <u at pkh.me> wrote:

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


cool, didn't know that worked!


> > +    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.
>
> yeah I'll just drop the log line, and send a new patch.


More information about the ffmpeg-devel mailing list