EFI_STATUS Status;\r
UINT32 DependenciesSize;\r
\r
+ if (Private == NULL) {\r
+ DEBUG ((DEBUG_ERROR, "FmpDxe(%s): PopulateDescriptor() - Private is NULL.\n", mImageIdName));\r
+ return;\r
+ }\r
+\r
if (Private->DescriptorPopulated) {\r
return;\r
}\r
\r
Status = EFI_SUCCESS;\r
\r
+ if (This == NULL) {\r
+ DEBUG ((DEBUG_ERROR, "FmpDxe(%s): GetImageInfo() - This is NULL.\n", mImageIdName));\r
+ Status = EFI_INVALID_PARAMETER;\r
+ goto cleanup;\r
+ }\r
+\r
//\r
// Retrieve the private context structure\r
//\r
\r
Status = EFI_SUCCESS;\r
\r
+ if (This == NULL) {\r
+ DEBUG ((DEBUG_ERROR, "FmpDxe(%s): GetImage() - This is NULL.\n", mImageIdName));\r
+ Status = EFI_INVALID_PARAMETER;\r
+ goto cleanup;\r
+ }\r
+\r
//\r
// Retrieve the private context structure\r
//\r
@param[in] Image Pointer to the image.\r
@param[in] ImageSize Size of the image.\r
@param[in] AdditionalHeaderSize Size of any headers that cannot be calculated by this function.\r
- @param[out] PayloadSize\r
+ @param[out] PayloadSize An optional pointer to a UINTN that holds the size of the payload\r
+ (image size minus headers)\r
\r
@retval !NULL Valid pointer to the header.\r
@retval NULL Structure is bad and pointer cannot be found.\r
IN CONST EFI_FIRMWARE_IMAGE_AUTHENTICATION *Image,\r
IN CONST UINTN ImageSize,\r
IN CONST UINTN AdditionalHeaderSize,\r
- OUT UINTN *PayloadSize\r
+ OUT UINTN *PayloadSize OPTIONAL\r
)\r
{\r
//\r
return NULL;\r
}\r
\r
- *PayloadSize = ImageSize - (sizeof (Image->MonotonicCount) + Image->AuthInfo.Hdr.dwLength + AdditionalHeaderSize);\r
+ if (PayloadSize != NULL) {\r
+ *PayloadSize = ImageSize - (sizeof (Image->MonotonicCount) + Image->AuthInfo.Hdr.dwLength + AdditionalHeaderSize);\r
+ }\r
+\r
return (VOID *)((UINT8 *)Image + sizeof (Image->MonotonicCount) + Image->AuthInfo.Hdr.dwLength + AdditionalHeaderSize);\r
}\r
\r
{\r
UINT32 CalculatedSize;\r
\r
+ if (Image == NULL) {\r
+ DEBUG ((DEBUG_ERROR, "FmpDxe(%s): GetAllHeaderSize() - Image is NULL.\n", mImageIdName));\r
+ return 0;\r
+ }\r
+\r
CalculatedSize = sizeof (Image->MonotonicCount) +\r
AdditionalHeaderSize +\r
Image->AuthInfo.Hdr.dwLength;\r
return EFI_UNSUPPORTED;\r
}\r
\r
+ if (This == NULL) {\r
+ DEBUG ((DEBUG_ERROR, "FmpDxe(%s): CheckImage() - This is NULL.\n", mImageIdName));\r
+ Status = EFI_INVALID_PARAMETER;\r
+ goto cleanup;\r
+ }\r
+\r
//\r
// Retrieve the private context structure\r
//\r
if (ImageIndex != 1) {\r
DEBUG ((DEBUG_ERROR, "FmpDxe(%s): CheckImage() - Image Index Invalid.\n", mImageIdName));\r
*ImageUpdatable = IMAGE_UPDATABLE_INVALID_TYPE;\r
- Status = EFI_SUCCESS;\r
+ Status = EFI_INVALID_PARAMETER;\r
goto cleanup;\r
}\r
\r
return EFI_UNSUPPORTED;\r
}\r
\r
+ if (This == NULL) {\r
+ DEBUG ((DEBUG_ERROR, "FmpDxe(%s): SetTheImage() - This is NULL.\n", mImageIdName));\r
+ Status = EFI_INVALID_PARAMETER;\r
+ goto cleanup;\r
+ }\r
+\r
//\r
// Retrieve the private context structure\r
//\r
EFI_STATUS Status;\r
FIRMWARE_MANAGEMENT_PRIVATE_DATA *Private;\r
\r
+ if (Context == NULL) {\r
+ ASSERT (Context != NULL);\r
+ return;\r
+ }\r
+\r
Private = (FIRMWARE_MANAGEMENT_PRIVATE_DATA *)Context;\r
\r
if (!Private->FmpDeviceLocked) {\r