[FFmpeg-devel] libavutil/encryption_info: Allow multiple init info.

Jacob Trimble modmaker at google.com
Sat Apr 28 03:30:46 EEST 2018


On Fri, Apr 27, 2018 at 10:33 AM, Jacob Trimble <modmaker at google.com> wrote:
> On Mon, Apr 23, 2018 at 11:03 AM, Jacob Trimble <modmaker at google.com> wrote:
>> While integrating my encryption info changes, I noticed a problem with
>> the init info structs.  I implemented them as side-data on the Stream.
>> But this means there can only be one per stream.  However, there can
>> be multiple 'pssh' atoms in a single stream (e.g. for key rotation or
>> multiple key systems). (sorry for not noticing sooner)
>>
>> Attached is a patch to fix this by making the init info a
>> singly-linked-list.  It is ABI compatible and is still easy to use and
>> understand.  The alternative would be to break ABI and have the
>> side-data methods return an array of pointers.  I could do that
>> instead if that is preferable.
>
> Ping

Noticed a bug, fixed.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-libavutil-encryption_info-Allow-multiple-init-info-v2.patch
Type: text/x-patch
Size: 8659 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180427/779e942a/attachment.bin>


More information about the ffmpeg-devel mailing list