[FFmpeg-devel] support for reading / writing encrypted MP4 files
michael at niedermayer.cc
Tue Dec 22 14:02:58 CET 2015
On Tue, Dec 22, 2015 at 07:36:41AM +0000, Eran Kornblau wrote:
> > > You're right, didn't know that.
> > > Please let me know if there are any changes you want me to apply in order to push the third one (decryption support).
> > sounds like you missed:
> > http://ffmpeg.org/pipermail/ffmpeg-devel/2015-December/185235.html
> > http://ffmpeg.org/pipermail/ffmpeg-devel/2015-December/185238.html
> Correct, I missed it. It's just that I'm not getting any emails from this mailing list
> (verified they are not getting to the junk folder...).
you are not subscribed (with this email address you used here at least
> So I've been polling the page of the last message of this thread in ffmpeg.org,
> and guess these posts were added to a different leaf of the message tree...
> Would really appreciate it if this could be resolved, either add my work email -
> eran dot kornblau at kaltura dot com or my personal email -
> erankor at gmail dot com.
> > if you want to maintain the code in the future then please send a
> > patch that adds you to the maintainers file
> > > + 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
> I see options to address this:
> 1. overwrite the codec_id only when it's AV_CODEC_ID_NONE
> 2. (stricter) save the format 4cc (that is loaded in ff_mov_read_stsd_entries) to some member of
> MOVStreamContext, and overwrite the codec id only when the format is 'encv' or 'enca'
> please let me know which one you prefer, and I will update the patch.
i think the strcter variant sounds more robust
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
DNS cache poisoning attacks, popular search engine, Google internet authority
dont be evil, please
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: Digital signature
More information about the ffmpeg-devel