summaryrefslogtreecommitdiff
path: root/vp9
diff options
context:
space:
mode:
authorYaowu Xu <yaowu@google.com>2014-05-30 10:15:30 -0700
committerYaowu Xu <yaowu@google.com>2014-05-30 10:18:54 -0700
commit538af7db5ffe2ab560511a3bc0ae74dae9316c14 (patch)
tree8f892bfb87713868f6de175e1590d2d121da9cf9 /vp9
parentd009c2360e899c13690dae679787243481d56379 (diff)
downloadlibvpx-538af7db5ffe2ab560511a3bc0ae74dae9316c14.tar
libvpx-538af7db5ffe2ab560511a3bc0ae74dae9316c14.tar.gz
libvpx-538af7db5ffe2ab560511a3bc0ae74dae9316c14.tar.bz2
libvpx-538af7db5ffe2ab560511a3bc0ae74dae9316c14.zip
Fix a problem of using an uninitialized parameter
This commit added a call to set speed feature before initializing motion search, fixed the problem where unintialized search method is used before its value being set. Change-Id: I537e4612bf0d00fd6f51396fd222d4b3bd6fde58
Diffstat (limited to 'vp9')
-rw-r--r--vp9/encoder/vp9_encoder.c14
1 files changed, 5 insertions, 9 deletions
diff --git a/vp9/encoder/vp9_encoder.c b/vp9/encoder/vp9_encoder.c
index 90155f3a9..4170fbecc 100644
--- a/vp9/encoder/vp9_encoder.c
+++ b/vp9/encoder/vp9_encoder.c
@@ -486,15 +486,11 @@ static void update_frame_size(VP9_COMP *cpi) {
vpx_internal_error(&cm->error, VPX_CODEC_MEM_ERROR,
"Failed to reallocate scaled last source buffer");
- {
- int y_stride = cpi->scaled_source.y_stride;
-
- if (cpi->sf.search_method == NSTEP) {
- vp9_init3smotion_compensation(&cpi->ss_cfg, y_stride);
- } else if (cpi->sf.search_method == DIAMOND) {
- vp9_init_dsmotion_compensation(&cpi->ss_cfg, y_stride);
- }
- }
+ set_speed_features(cpi);
+ if (cpi->sf.search_method == NSTEP)
+ vp9_init3smotion_compensation(&cpi->ss_cfg, cpi->scaled_source.y_stride);
+ else if (cpi->sf.search_method == DIAMOND)
+ vp9_init_dsmotion_compensation(&cpi->ss_cfg, cpi->scaled_source.y_stride);
init_macroblockd(cm, xd);
}