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

Jacob Trimble modmaker at google.com
Mon May 7 19:34:16 EEST 2018


On Fri, Apr 27, 2018 at 5:30 PM, Jacob Trimble <modmaker at google.com> wrote:
> 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.

Ping.


More information about the ffmpeg-devel mailing list