Re: [PATCH i-g-t] Avoid crash from uninitialized pipe_crc pointer

["Reddy Guddati, Santhosh" <santhosh.reddy.guddati@intel.com>]

Hi Pranay,

Please add closes tag if issue is known otherwise, LGTM,
Reviewed-by: Santhosh Reddy Guddati <santhosh.reddy.guddati@intel.com>

On 06-01-2026 14:19, Pranay Samala wrote:
> The data_t structure was not initialized, leaving pipe_crc with an
> undefined value on first use and could result in freeing an invalid
> pointer.
> Initialize data_t to zero so pipe_crc starts as NULL and the
> cleanup path safely manages the CRC object.
> 
> Signed-off-by: Pranay Samala <pranay.samala@intel.com>
> ---
>   tests/intel/kms_fb_coherency.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/intel/kms_fb_coherency.c b/tests/intel/kms_fb_coherency.c
> index 59bc29d19..ca915bc16 100644
> --- a/tests/intel/kms_fb_coherency.c
> +++ b/tests/intel/kms_fb_coherency.c
> @@ -238,7 +238,7 @@ static void select_valid_pipe_output_combo(data_t *data)
>   
>   int igt_main()
>   {
> -	data_t data;
> +	data_t data = { 0 };
>   
>   	igt_fixture() {
>   		data.drm_fd = drm_open_driver_master(DRIVER_INTEL | DRIVER_XE);