Re: [igt-dev] [PATCH i-g-t] tests/kms_hdr: Add negative test for invalid metadata sizes

["Shankar, Uma" <uma.shankar@intel.com>]

> -----Original Message-----
> From: igt-dev <igt-dev-bounces@lists.freedesktop.org> On Behalf Of Swati Sharma
> Sent: Monday, December 26, 2022 4:09 PM
> To: igt-dev@lists.freedesktop.org
> Subject: [igt-dev] [PATCH i-g-t] tests/kms_hdr: Add negative test for invalid
> metadata sizes
> 
> Negative test is added in which HDR output metadata prop is set with invalid sizes.
> 
> Signed-off-by: Swati Sharma <swati2.sharma@intel.com>
> ---
>  tests/kms_hdr.c | 32 ++++++++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
> 
> diff --git a/tests/kms_hdr.c b/tests/kms_hdr.c index 655c3e14..629a45a7 100644
> --- a/tests/kms_hdr.c
> +++ b/tests/kms_hdr.c
> @@ -50,6 +50,7 @@ enum {
>  	TEST_DPMS = 1 << 1,
>  	TEST_SUSPEND = 1 << 2,
>  	TEST_SWAP = 1 << 3,
> +	TEST_INVALID_METADATA_SIZES = 1 << 4,
>  };
> 
>  /* BPC connector state. */
> @@ -327,6 +328,18 @@ static void set_hdr_output_metadata(data_t *data,
>  				     meta ? sizeof(*meta) : 0);
>  }
> 
> +/* Sets the HDR output metadata prop with invalid size. */ static int
> +set_invalid_hdr_output_metadata(data_t *data,
> +					   struct hdr_output_metadata const
> *meta,
> +					   size_t length)
> +{
> +	igt_output_replace_prop_blob(data->output,
> +				     IGT_CONNECTOR_HDR_OUTPUT_METADATA,
> meta,
> +				     meta ? length : 0);
> +
> +	return igt_display_try_commit_atomic(&data->display,
> +DRM_MODE_ATOMIC_ALLOW_MODESET, NULL); }
> +
>  /* Converts a double to 861-G spec FP format. */  static uint16_t
> calc_hdr_float(double val)  { @@ -518,6 +531,19 @@ static void
> test_static_swap(data_t *data, enum pipe pipe, igt_output_t *output)
>  	igt_remove_fb(data->fd, &afb);
>  }
> 
> +static void test_invalid_metadata_sizes(data_t *data, igt_output_t
> +*output) {
> +	struct hdr_output_metadata hdr;
> +	size_t metadata_size = sizeof(&hdr);
> +
> +	fill_hdr_output_metadata_st2048(&hdr);
> +
> +	igt_assert_eq(set_invalid_hdr_output_metadata(data, &hdr, 1), -EINVAL);
> +	igt_assert_eq(set_invalid_hdr_output_metadata(data, &hdr, metadata_size
> + 1), -EINVAL);
> +	igt_assert_eq(set_invalid_hdr_output_metadata(data, &hdr, metadata_size
> - 1), -EINVAL);
> +	igt_assert_eq(set_invalid_hdr_output_metadata(data, &hdr,
> +metadata_size + sizeof(struct hdr_output_metadata)), -EINVAL); 

I guess 2 invalid sizes should be enough. Other that that, looks good to me.
Reviewed-by: Uma Shankar <uma.shankar@intel.com>

}
> +
>  /* Returns true if an output supports HDR metadata property. */  static bool
> has_hdr(igt_output_t *output)  { @@ -565,6 +591,8 @@ static void test_hdr(data_t
> *data, uint32_t flags)
>  						test_static_toggle(data, pipe,
> output, flags);
>  					if (flags & TEST_SWAP)
>  						test_static_swap(data, pipe,
> output);
> +					if (flags &
> TEST_INVALID_METADATA_SIZES)
> +						test_invalid_metadata_sizes(data,
> output);
>  				}
> 
>  				/* One pipe is enough */
> @@ -613,6 +641,10 @@ igt_main
>  	igt_subtest_with_dynamic("static-swap")
>  		test_hdr(&data, TEST_SWAP);
> 
> +	igt_describe("Tests invalid HDR metadata sizes");
> +	igt_subtest_with_dynamic("invalid-metadata-sizes")
> +		test_hdr(&data, TEST_INVALID_METADATA_SIZES);
> +
>  	igt_fixture {
>  		igt_display_fini(&data.display);
>  		close(data.fd);
> --
> 2.25.1