]> git.sesse.net Git - ffmpeg/commit
avformat/dashdec: drop arbitrary DASH manifest size limit
authorJan Ekström <jeebjp@gmail.com>
Thu, 3 Sep 2020 16:50:08 +0000 (19:50 +0300)
committerJan Ekström <jeebjp@gmail.com>
Sat, 5 Sep 2020 08:50:49 +0000 (11:50 +0300)
commit3249c757aed678780e22e99a1a49f4672851bca9
tree08a09d682e828885c0fed9ec18cc8803ddda5cf3
parentd263838958def82118bc2abe7fe1ccb0821af550
avformat/dashdec: drop arbitrary DASH manifest size limit

Currently the utilized AVBPrint API is internally limited to unsigned
integers, so if we limit the file size as well as the amount to read
to UINT_MAX - 1, we do not require additional limiting to be performed
on the values.

This change is based on the fact that initially the 8*1024 value added
in 96d70694aea64616c68db8be306c159c73fb3980 was only for the case where
the file size was not known. It was not a maximum file size limit.

In 29121188983932f79aef8501652630d322a9974c this was reworked to be
a maximum manifest file size limit, while its commit message appears
to only note that it added support for larger manifest file sizes.

This should enable various unfortunately large MPEG-DASH manifests,
such as Youtube's multi-megabyte live stream archives to load up
as well as bring back the original intent of the logic.
libavformat/dashdec.c