[FFmpeg-devel] [PATCH v2 01/14] h264_mp4toannexb: Remove unnecessary check

Michael Niedermayer michael at niedermayer.cc
Mon Dec 16 00:56:00 EET 2019


On Sat, Dec 14, 2019 at 11:19:13PM +0100, Andreas Rheinhardt wrote:
> There can be at most 31 SPS and 255 PPS in the mp4/Matroska extradata.
> Given that each has a size of at most 2^16-1, the length of the output
> derived from these parameter sets can never overflow an ordinary 32 bit
> integer. So use a simple uint32_t instead of uint64_t and replace the
> unnecessary check with an av_assert1.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
>  libavcodec/h264_mp4toannexb_bsf.c | 10 +++-------
>  1 file changed, 3 insertions(+), 7 deletions(-)

will apply

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20191215/3b2645eb/attachment.sig>


More information about the ffmpeg-devel mailing list