diff options
author | Victorien Le Couviour--Tuffet <victorien@videolan.org> | 2022-05-31 17:23:56 +0300 |
---|---|---|
committer | Ronald S. Bultje <rsbultje@gmail.com> | 2022-05-31 19:28:34 +0300 |
commit | b4f9eac85803303760f90ebad49de068cc2663d7 (patch) | |
tree | 0c340acf4ddb0a77e48c0f8d84bf592153f376cf /tests | |
parent | 8c079f784a48e7241a1bb6cca0a063b7a654cd2b (diff) |
checkasm: Fix uninitialized variable
fg_data->num_y_points is used in generate_grain_uv, but is only set
after the call: move the initialization above.
Diffstat (limited to 'tests')
-rw-r--r-- | tests/checkasm/filmgrain.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/checkasm/filmgrain.c b/tests/checkasm/filmgrain.c index 9aa4f26..638e83f 100644 --- a/tests/checkasm/filmgrain.c +++ b/tests/checkasm/filmgrain.c @@ -283,6 +283,7 @@ static void check_fguv_sbrow(const Dav1dFilmGrainDSPContext *const dsp) { fg_data[0].grain_scale_shift = rnd() & 3; fg_data[0].ar_coeff_shift = (rnd() & 3) + 6; fg_data[0].ar_coeff_lag = rnd() & 3; + fg_data[0].num_y_points = csfl ? 2 + (rnd() % 13) : 0; const int num_y_pos = 2 * fg_data[0].ar_coeff_lag * (fg_data[0].ar_coeff_lag + 1); for (int n = 0; n < num_y_pos; n++) fg_data[0].ar_coeffs_y[n] = (rnd() & 0xff) - 128; @@ -294,7 +295,6 @@ static void check_fguv_sbrow(const Dav1dFilmGrainDSPContext *const dsp) { fg_data, uv_pl HIGHBD_TAIL_SUFFIX); if (csfl) { - fg_data[0].num_y_points = 2 + (rnd() % 13); const int pad = 0xff / fg_data[0].num_y_points; for (int n = 0; n < fg_data[0].num_y_points; n++) { fg_data[0].y_points[n][0] = 0xff * n / fg_data[0].num_y_points; |