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

Michael Niedermayer michaelni at gmx.at
Fri Mar 6 01:37:30 EET 2020


On Sun, Dec 15, 2019 at 11:56:00PM +0100, Michael Niedermayer wrote:
> 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

will apply the rest of the patchset

thx

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

Dictatorship: All citizens are under surveillance, all their steps and
actions recorded, for the politicians to enforce control.
Democracy: All politicians are under surveillance, all their steps and
actions recorded, for the citizens to enforce control.
-------------- 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/20200306/8cd3c963/attachment.sig>


More information about the ffmpeg-devel mailing list