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.
pgptTgyHuyUMB.pgp
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel