[FFmpeg-devel] [PATCH v1] avcodec/h264_parse: decode sps before pps
Fu, Linjie
linjie.fu at intel.com
Tue Oct 8 05:50:43 EEST 2019
> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of Jun
> Li
> Sent: Tuesday, October 8, 2019 05:42
> To: ffmpeg-devel at ffmpeg.org
> Subject: [FFmpeg-devel] [PATCH v1] avcodec/h264_parse: decode sps
> before pps
>
> Fix ticket #6422
> The content put pps before sps, which is not a common case. The change
> just put decoding sps before pps, just for compability.
>
> Signed-off-by: Jun Li <junli1026 at gmail.com>
> ---
> libavcodec/h264_parse.c | 21 +++++++++++----------
> 1 file changed, 11 insertions(+), 10 deletions(-)
>
> diff --git a/libavcodec/h264_parse.c b/libavcodec/h264_parse.c
> index 352ffea948..26aaa751ad 100644
> --- a/libavcodec/h264_parse.c
> +++ b/libavcodec/h264_parse.c
> @@ -373,35 +373,36 @@ static int decode_extradata_ps(const uint8_t *data,
> int size, H264ParamSets *ps,
> goto fail;
> }
>
> - for (i = 0; i < pkt.nb_nals; i++) {
> + /* decode sps before pps, just in case some bitstream put pps before sps.
> */
> + for(i = 0; i < pkt.nb_nals; i++) {
Nit: Missing space here.
- linjie
More information about the ffmpeg-devel
mailing list