Fix dead code in SpdmMeasurement

Signed-off-by: Wenxing Hou <wenxing.hou@intel.com>
tianocore · Aug 18, 2023 · aeb8c8c · aeb8c8c
1 parent 2b5bbaf
commit aeb8c8c
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/SecurityPkg/DeviceSecurity/SpdmSecurityLib/SpdmMeasurement.c b/SecurityPkg/DeviceSecurity/SpdmSecurityLib/SpdmMeasurement.c
@@ -653,12 +653,14 @@ DoDeviceMeasurement (
                      NULL, 0
                      );
       if (LIBSPDM_STATUS_IS_ERROR (SpdmReturn)) {
-        continue;
-      } else if (SpdmReturn == LIBSPDM_STATUS_VERIF_FAIL) {
-        AuthState                       = TCG_DEVICE_SECURITY_EVENT_DATA_DEVICE_AUTH_STATE_FAIL_INVALID;
-        SecurityState->MeasurementState = EDKII_DEVICE_SECURITY_STATE_ERROR_DEVICE_ERROR;
-        Status                          = ExtendMeasurement (SpdmDeviceContext, AuthState, 0, NULL, NULL, NULL, SecurityState);
-        continue;
+        if (SpdmReturn == LIBSPDM_STATUS_VERIF_FAIL) {
+          AuthState                       = TCG_DEVICE_SECURITY_EVENT_DATA_DEVICE_AUTH_STATE_FAIL_INVALID;
+          SecurityState->MeasurementState = EDKII_DEVICE_SECURITY_STATE_ERROR_DEVICE_ERROR;
+          Status                          = ExtendMeasurement (SpdmDeviceContext, AuthState, 0, NULL, NULL, NULL, SecurityState);
+          continue;
+        } else {
+          continue;
+        }
       }
 
       if (  (ReceivedNumberOfBlock == NumberOfBlocks - 1)