diff options
Diffstat (limited to 'media/libvpx/input_frame_validation_vp9.patch')
| -rw-r--r-- | media/libvpx/input_frame_validation_vp9.patch | 44 |
1 files changed, 44 insertions, 0 deletions
diff --git a/media/libvpx/input_frame_validation_vp9.patch b/media/libvpx/input_frame_validation_vp9.patch new file mode 100644 index 0000000000..bcbfc9b4c2 --- /dev/null +++ b/media/libvpx/input_frame_validation_vp9.patch @@ -0,0 +1,44 @@ +# HG changeset patch +# User Randell Jesup <rjesup@jesup.org> +# Parent 87841f3bfc9d99a37e31cd43b2e2d03c325af84f +Bug 1315288: Add input checks for VP9 r=rillian + +diff --git a/vp9/vp9_cx_iface.c b/vp9/vp9_cx_iface.c +--- a/vp9/vp9_cx_iface.c ++++ b/vp9/vp9_cx_iface.c +@@ -1353,21 +1353,30 @@ static vpx_codec_err_t encoder_encode(vp + cpi->svc.time_stamp_superframe = dst_time_stamp; + + // Set up internal flags + if (ctx->base.init_flags & VPX_CODEC_USE_PSNR) cpi->b_calculate_psnr = 1; + + if (img != NULL) { + res = image2yuvconfig(img, &sd); + +- // Store the original flags in to the frame buffer. Will extract the +- // key frame flag when we actually encode this frame. +- if (vp9_receive_raw_frame(cpi, flags | ctx->next_frame_flags, &sd, +- dst_time_stamp, dst_end_time_stamp)) { +- res = update_error_state(ctx, &cpi->common.error); ++ if (sd.y_width != ctx->cfg.g_w || sd.y_height != ctx->cfg.g_h) { ++ /* from vpx_encoder.h for g_w/g_h: ++ "Note that the frames passed as input to the encoder must have this ++ resolution" ++ */ ++ ctx->base.err_detail = "Invalid input frame resolution"; ++ res = VPX_CODEC_INVALID_PARAM; ++ } else { ++ // Store the original flags in to the frame buffer. Will extract the ++ // key frame flag when we actually encode this frame. ++ if (vp9_receive_raw_frame(cpi, flags | ctx->next_frame_flags, &sd, ++ dst_time_stamp, dst_end_time_stamp)) { ++ res = update_error_state(ctx, &cpi->common.error); ++ } + } + ctx->next_frame_flags = 0; + } + + cx_data = ctx->cx_data; + cx_data_sz = ctx->cx_data_sz; + + /* Any pending invisible frames? */ |