Check VideoFrame::IsValidConfig() prior to allocating buffers for FFmpeg.

media/filters/ffmpeg_video_decoder.cc

Index: media/filters/ffmpeg_video_decoder.cc
diff --git a/media/filters/ffmpeg_video_decoder.cc b/media/filters/ffmpeg_video_decoder.cc
index c2d34c28d0b77d57d31f5a8121c75ca587c6ef7e..7be2baf2cc937560e90ddf0033acef3604ea8c0a 100644
--- a/media/filters/ffmpeg_video_decoder.cc
+++ b/media/filters/ffmpeg_video_decoder.cc
@@ -81,6 +81,9 @@ int FFmpegVideoDecoder::GetVideoBuffer(AVCodecContext* codec_context,
   if ((ret = av_image_check_size(width, height, 0, NULL)) < 0)
     return ret;
 
+  if (!VideoFrame::IsValidConfig(format, width, height))
+    return AVERROR(EINVAL);
+
   scoped_refptr<VideoFrame> video_frame =
       VideoFrame::CreateFrame(format, width, height, kNoTimestamp());

[scherkus (not reviewing), 2012/08/06 20:42:34]

this DCHECKs for IsValidConfig() but in release mode we'll end up creating
invalid frames