Re: [FFmpeg-devel] [PATCH 1/3] avformat/matroskadec: Sanitize SeekHead entries

2020-05-08 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Andreas Rheinhardt: >> A Seek element in a Matroska SeekHead should contain a SeekID and a >> SeekPosition element and upon reading, they should be sanitized: >> >> Given that IDs are restricted to 32 bit, longer SeekIDs should be treated >> as invalid. Instead currently the l

Re: [FFmpeg-devel] [PATCH 1/3] avformat/matroskadec: Sanitize SeekHead entries

2020-05-06 Thread Andreas Rheinhardt
Andreas Rheinhardt: > A Seek element in a Matroska SeekHead should contain a SeekID and a > SeekPosition element and upon reading, they should be sanitized: > > Given that IDs are restricted to 32 bit, longer SeekIDs should be treated > as invalid. Instead currently the lower 32 bits have been use

[FFmpeg-devel] [PATCH 1/3] avformat/matroskadec: Sanitize SeekHead entries

2020-04-30 Thread Andreas Rheinhardt
A Seek element in a Matroska SeekHead should contain a SeekID and a SeekPosition element and upon reading, they should be sanitized: Given that IDs are restricted to 32 bit, longer SeekIDs should be treated as invalid. Instead currently the lower 32 bits have been used. For SeekPosition, no check