Hi Zhichao,
I think you're right. I will submit a v2 patch that minimizes the amount of
pointer validation required for parsing the remaining tables.
Kind regards,
Krzysztof
-Original Message-
From: Gao, Zhichao
Sent: Friday, August 16, 2019 8:35
To: Krzysztof Koch ; devel@edk2.group
For FadtMinorRevision and X_DsdtAddress, I don't think they are required
section. Maybe we should consider check the length before check them. As I
know, the OVMF's FACP table doesn't have the section after flag.
Thanks,
Zhichao
> -Original Message-
> From: Krzysztof Koch [mailto:krzysz
Check if global pointers have been successfully updated before they
are used for further table parsing.
Signed-off-by: Krzysztof Koch
---
Notes:
v1:
- Test against NULL pointers [Krzysztof]
ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Fadt/FadtParser.c | 14
++
1 f