From 727fb9b1d3a076f4ecd46dab97ec64a114ac89ab Mon Sep 17 00:00:00 2001 From: mdkinney Date: Sat, 9 May 2009 00:45:25 +0000 Subject: 1) Clarify file and function header comments for the PE/COFF Extra Action library class 2) Update the file and function header comments in the BasePeCoffExtraActionLibNull 3) Add ASSERT() statements to APIs in BasePeCoffExtraActionLibNull is ImageContext is NULL. git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@8272 6f19259b-4bc3-4df7-8a09-765794883524 --- .../BasePeCoffExtraActionLibNull.inf | 5 +++- .../PeCoffExtraActionLib.c | 28 ++++++++++------------ 2 files changed, 16 insertions(+), 17 deletions(-) (limited to 'MdePkg/Library/BasePeCoffExtraActionLibNull') diff --git a/MdePkg/Library/BasePeCoffExtraActionLibNull/BasePeCoffExtraActionLibNull.inf b/MdePkg/Library/BasePeCoffExtraActionLibNull/BasePeCoffExtraActionLibNull.inf index 50b5bb37be..de951da269 100644 --- a/MdePkg/Library/BasePeCoffExtraActionLibNull/BasePeCoffExtraActionLibNull.inf +++ b/MdePkg/Library/BasePeCoffExtraActionLibNull/BasePeCoffExtraActionLibNull.inf @@ -29,7 +29,10 @@ [Sources.common] PeCoffExtraActionLib.c - [Packages] MdePkg/MdePkg.dec +[LibraryClasses.common] + DebugLib + + diff --git a/MdePkg/Library/BasePeCoffExtraActionLibNull/PeCoffExtraActionLib.c b/MdePkg/Library/BasePeCoffExtraActionLibNull/PeCoffExtraActionLib.c index 47bb0db891..6f5200361d 100644 --- a/MdePkg/Library/BasePeCoffExtraActionLibNull/PeCoffExtraActionLib.c +++ b/MdePkg/Library/BasePeCoffExtraActionLibNull/PeCoffExtraActionLib.c @@ -14,17 +14,15 @@ #include #include +#include /** - Applies additional actions to relocate fixups to a PE/COFF image. + Performs additional actions after a PE/COFF image has been loaded and relocated. - Generally this function is called after sucessfully Applying relocation fixups - to a PE/COFF image for some specicial purpose. - As a example, For NT32 emulator, the function should be implemented and called - to support source level debug. - - @param ImageContext Pointer to the image context structure that describes the PE/COFF - image that is being relocated. + If ImageContext is NULL, then ASSERT(). + + @param ImageContext Pointer to the image context structure that describes the + PE/COFF image that has already been loaded and relocated. **/ VOID @@ -33,20 +31,17 @@ PeCoffLoaderRelocateImageExtraAction ( IN OUT PE_COFF_LOADER_IMAGE_CONTEXT *ImageContext ) { + ASSERT (ImageContext != NULL); } /** - Unloads a loaded PE/COFF image from memory and releases its taken resource. - - Releases any environment specific resources that were allocated when the image - specified by ImageContext was loaded using PeCoffLoaderLoadImage(). - For NT32 emulator, the PE/COFF image loaded by system needs to release. - For real platform, the PE/COFF image loaded by Core doesn't needs to be unloaded, + Performs additional actions just before a PE/COFF image is unloaded. Any resources + that were allocated by PeCoffLoaderRelocateImageExtraAction() must be freed. If ImageContext is NULL, then ASSERT(). - @param ImageContext Pointer to the image context structure that describes the PE/COFF - image to be unloaded. + @param ImageContext Pointer to the image context structure that describes the + PE/COFF image that is being unloaded. **/ VOID @@ -55,4 +50,5 @@ PeCoffLoaderUnloadImageExtraAction ( IN OUT PE_COFF_LOADER_IMAGE_CONTEXT *ImageContext ) { + ASSERT (ImageContext != NULL); } -- cgit v1.2.3