[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <839a74c1-6e08-441b-a6e3-9923dfc3f303@amd.com>
Date: Wed, 6 Mar 2024 10:39:59 +0530
From: Shyam Sundar S K <Shyam-sundar.S-k@....com>
To: Armin Wolf <W_Armin@....de>
Cc: hdegoede@...hat.com, ilpo.jarvinen@...ux.intel.com,
platform-driver-x86@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 0/4] platform/x86/amd/pmf: Fix policy binary handling
On 3/5/2024 02:20, Armin Wolf wrote:
> This patch series fixes various issues inside the policy binary
> handling code.
> The first patch makes sure that a valid error code is returned upon
> failing to start the policy engine, while the second patch drops the
> usage of readl() on non-io memory.
> The last two patches fix a possible out-of-bounds memory access when
> parsing the policy binary header.
>
> All patches are compile-tested only.
>
> Changes since v2:
> - add patches 1 and 3
>
> Changes since v1:
> - get the full dword instead of only 8 bits when reading the header
> - check if the policy buffer also has enough room for storing the length
Thank you Armin. Series looks good to me.
Reviewed-by: Shyam Sundar S K Shyam-sundar.S-k@....com
>
> Armin Wolf (4):
> platform/x86/amd/pmf: Fix return value of
> amd_pmf_start_policy_engine()
> platform/x86/amd/pmf: Do not use readl() for policy buffer access
> platform/x86/amd/pmf: Use struct for cookie header
> platform/x86/amd/pmf: Fix possible out-of-bound memory accesses
>
> drivers/platform/x86/amd/pmf/pmf.h | 6 +++++-
> drivers/platform/x86/amd/pmf/tee-if.c | 21 +++++++++++++--------
> 2 files changed, 18 insertions(+), 9 deletions(-)
>
> --
> 2.39.2
>
Powered by blists - more mailing lists