[FFmpeg-devel] [PATCH] h264_ps: validate chroma sample location

Andreas Cadhalpun andreas.cadhalpun at googlemail.com
Fri Jan 6 22:44:16 EET 2017


Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun at googlemail.com>
---
 libavcodec/h264_ps.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c
index 8218e3a010..089bfc650a 100644
--- a/libavcodec/h264_ps.c
+++ b/libavcodec/h264_ps.c
@@ -181,6 +181,10 @@ static inline int decode_vui_parameters(GetBitContext *gb, AVCodecContext *avctx
     if (get_bits1(gb)) {
         /* chroma_sample_location_type_top_field */
         avctx->chroma_sample_location = get_ue_golomb(gb) + 1;
+        if (!av_chroma_location_name(avctx->chroma_sample_location)) {
+            av_log(avctx, AV_LOG_WARNING, "Invalid chroma sample location %d, setting to unspecified\n", avctx->chroma_sample_location);
+            avctx->chroma_sample_location = AVCHROMA_LOC_UNSPECIFIED;
+        }
         get_ue_golomb(gb);  /* chroma_sample_location_type_bottom_field */
     }
 
-- 
2.11.0


More information about the ffmpeg-devel mailing list