diff options
| author | Sami Mujawar <sami.mujawar@arm.com> | 2020-09-03 11:45:07 +0100 |
|---|---|---|
| committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2020-10-21 13:33:03 +0000 |
| commit | 653fb710ce2de21544e38ae81941eba01c94426a (patch) | |
| tree | 278246f853831ab02acf5c731f5aebd5a45f3116 /DynamicTablesPkg | |
| parent | 26442d11e620a9e81c019a24a4ff38441c64ba10 (diff) | |
| download | edk2-653fb710ce2de21544e38ae81941eba01c94426a.tar.gz edk2-653fb710ce2de21544e38ae81941eba01c94426a.tar.bz2 edk2-653fb710ce2de21544e38ae81941eba01c94426a.zip | |
DynamicTablesPkg: Fix order of assert checks
Reordered the asserts to first check if the pointer is valid
before de-referencing the pointer.
Signed-off-by: Sami Mujawar <sami.mujawar@arm.com>
Reviewed-by: Alexei Fedorov <Alexei.Fedorov@arm.com>
Diffstat (limited to 'DynamicTablesPkg')
| -rw-r--r-- | DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c index 2197e50dc7..a705d771be 100644 --- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c +++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c @@ -90,10 +90,10 @@ FreeSsdtSerialPortTableEx ( UINTN Index;
ASSERT (This != NULL);
+ ASSERT (AcpiTableInfo != NULL);
+ ASSERT (CfgMgrProtocol != NULL);
ASSERT (AcpiTableInfo->TableGeneratorId == This->GeneratorID);
ASSERT (AcpiTableInfo->AcpiTableSignature == This->AcpiTableSignature);
- ASSERT (CfgMgrProtocol != NULL);
- ASSERT (AcpiTableInfo != NULL);
if ((Table == NULL) ||
(*Table == NULL) ||
|