platform/x86/amd: hfi: Fix pcct_tbl leak in amd_hfi_metadata_parser()

JIRA: https://issues.redhat.com/browse/RHEL-107081

commit d3a8ca2
Author: Zhen Ni <zhen.ni@easystack.cn>
Date:   Fri Aug 22 16:33:29 2025 +0800

    platform/x86/amd: hfi: Fix pcct_tbl leak in amd_hfi_metadata_parser()

    Fix a permanent ACPI table memory leak when amd_hfi_metadata_parser()
    fails due to invalid PCCT table length or memory allocation errors.

    Fixes: d4e95ea ("platform/x86: hfi: Parse CPU core ranking data from shared memory")
    Cc: stable@vger.kernel.org
    Signed-off-by: Zhen Ni <zhen.ni@easystack.cn>
    Reviewed-by: Mario Limonciello (AMD) <superm1@kernel.org>
    Link: https://lore.kernel.org/r/20250822083329.710857-1-zhen.ni@easystack.cn
    Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
    Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>

Signed-off-by: Steve Best <sbest@redhat.com>

Author: sfbest

drivers/platform/x86/amd/hfi/hfi.c

@@ -385,19 +385,25 @@ static int amd_hfi_metadata_parser(struct platform_device *pdev,
 	amd_hfi_data->pcct_entry = pcct_entry;
 	pcct_ext = (struct acpi_pcct_ext_pcc_slave *)pcct_entry;
 
-	if (pcct_ext->length <= 0)
-		return -EINVAL;
+	if (pcct_ext->length <= 0) {
+		ret = -EINVAL;
+		goto out;
+	}
 
 	amd_hfi_data->shmem = devm_kzalloc(amd_hfi_data->dev, pcct_ext->length, GFP_KERNEL);
-	if (!amd_hfi_data->shmem)
-		return -ENOMEM;
+	if (!amd_hfi_data->shmem) {
+		ret = -ENOMEM;
+		goto out;
+	}
 
 	pcc_chan->shmem_base_addr = pcct_ext->base_address;
 	pcc_chan->shmem_size = pcct_ext->length;
 
 	/* parse the shared memory info from the PCCT table */
 	ret = amd_hfi_fill_metadata(amd_hfi_data);
 
+out:
+	/* Don't leak any ACPI memory */
 	acpi_put_table(pcct_tbl);
 
 	return ret;