On Fri, 18 Nov 2022 at 05:37, Mikhail Ilin <ilin.mikhail...@gmail.com> wrote:
>
> An incorrect 1st parameter is passed to the fix_member()
> function. Should use a pointer to the beginning of the parent structure
> (bpdt or subpart_dir, because are boxed), not to their fields. Otherwise,
> this leads to an overrun of the structure boundary, since in the
> fix_member() function, an 'offset' is made, relative to the 1st argument,
> which itself is an 'offset' from the beginning of the structure.
>
> Signed-off-by: Mikhail Ilin <ilin.mikhail...@gmail.com>
> ---
> tools/ifwitool.c | 44 +++++++++++++++++++-------------------------
> 1 file changed, 19 insertions(+), 25 deletions(-)
Reviewed-by: Simon Glass <s...@chromium.org>
Fixes: 56bf4f86307 ("x86: Add ifwitool for Intel Integrated Firmware Image")
