[FFmpeg-devel] support for reading / writing encrypted MP4 files

Michael Niedermayer michaelni at gmx.at
Tue Dec 15 14:57:45 CET 2015


On Tue, Dec 15, 2015 at 09:16:11AM +0000, Eran Kornblau wrote:
[...]

> @@ -4007,6 +4008,147 @@ static int mov_read_free(MOVContext *c, AVIOContext *pb, MOVAtom atom)
>      return 0;
>  }
>  
> +static int mov_read_frma(MOVContext *c, AVIOContext *pb, MOVAtom atom)
> +{
> +    uint32_t format = avio_rl32(pb);
> +    enum AVCodecID id;
> +    AVStream *st;
> +
> +    if (c->fc->nb_streams < 1)
> +        return 0;
> +    st = c->fc->streams[c->fc->nb_streams - 1];
> +
> +    id = mov_codec_id(st, format);
> +    st->codec->codec_id = id;

doesnt this allow changing the codec id to anything from anything ?
this seems potentially risky

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151215/51830263/attachment.sig>


More information about the ffmpeg-devel mailing list