[FFmpeg-devel] [PATCH] avformat/utils: Don't parse encrypted packets.

James Almer jamrial at gmail.com
Thu Aug 30 01:14:25 EEST 2018


On 8/29/2018 7:07 PM, Michael Niedermayer wrote:
> On Tue, Aug 28, 2018 at 10:58:43AM -0700, Jacob Trimble wrote:
>> If a packet is full-sample encrypted, then packet data can't be parsed
>> without decrypting it.  So this skips the packet parsing for those
>> packets.  If the packet has sub-sample encryption, it is assumed that
>> the headers are in the clear and the parser will only need that info.
>>
>> Signed-off-by: Jacob Trimble <modmaker at google.com>
>> ---
>>  libavformat/utils.c | 21 ++++++++++++++++++++-
>>  1 file changed, 20 insertions(+), 1 deletion(-)
> 
> Hmm, please correct me if iam wrong but IIUC
> 1. The demuxer has set need_parsing, indicating that it _requires_ a parser
> 2. Parsing cannot be done before decrypting
> 
> If all this and the set values are correct, logically, the fix would be
> to decrypt the packet before the parser not to skip the parser.

And if that can't be done, then the demuxer could perhaps set
st->need_parsing to 0 and skip parsing altogether?

> 
> Am i missing something ?
> 
> thx
> 
> [...]
> 
> 
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 



More information about the ffmpeg-devel mailing list