On 2013-03-04 11:37:27 +0100, Luca Barbato wrote: > The decoder assumes a single bitdepth for all the planes, the
bit depth > specification does allow that. not clear what the specification allows: "all the planes while the specification allows different bit depths for luma and chroma." > > Avoid the possible problems pointed in CVE-2013-2277 > > CC: [email protected] > --- > libavcodec/h264.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/libavcodec/h264.c b/libavcodec/h264.c > index a20a6f7..b11756e 100644 > --- a/libavcodec/h264.c > +++ b/libavcodec/h264.c > @@ -2747,6 +2747,12 @@ static int h264_set_parameter_from_sps(H264Context *h) > if (h->avctx->has_b_frames < 2) > h->avctx->has_b_frames = !h->low_delay; > > + if (h->sps.bit_depth_luma != h->sps.bit_depth_chroma) { > + av_log_missing_feature(h->avctx, > + "Different bit depth between chroma and luma", 1); > + return AVERROR(ENOSYS); despite my comment a patch would still be welcome Janne _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
