diff options
| author | eric_tian <eric_tian@6f19259b-4bc3-4df7-8a09-765794883524> | 2008-09-16 05:19:25 +0000 |
|---|---|---|
| committer | eric_tian <eric_tian@6f19259b-4bc3-4df7-8a09-765794883524> | 2008-09-16 05:19:25 +0000 |
| commit | 1349236994106e461f82b2df1df5ebbf1bd80898 (patch) | |
| tree | 1e20195dfcbe8ca4d76f49243d5d5a7368beb118 /MdeModulePkg/Core/Dxe/FwVol/FwVol.c | |
| parent | 9edc73ada43b313117ef1662c16c85114a480b8b (diff) | |
| download | edk2-1349236994106e461f82b2df1df5ebbf1bd80898.zip edk2-1349236994106e461f82b2df1df5ebbf1bd80898.tar.gz edk2-1349236994106e461f82b2df1df5ebbf1bd80898.tar.bz2 | |
follow up the comments from code review.
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@5891 6f19259b-4bc3-4df7-8a09-765794883524
Diffstat (limited to 'MdeModulePkg/Core/Dxe/FwVol/FwVol.c')
| -rw-r--r-- | MdeModulePkg/Core/Dxe/FwVol/FwVol.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/MdeModulePkg/Core/Dxe/FwVol/FwVol.c b/MdeModulePkg/Core/Dxe/FwVol/FwVol.c index c236282..16d66fa 100644 --- a/MdeModulePkg/Core/Dxe/FwVol/FwVol.c +++ b/MdeModulePkg/Core/Dxe/FwVol/FwVol.c @@ -83,6 +83,9 @@ GetFwVolHeader ( //
FvhLength = sizeof (EFI_FIRMWARE_VOLUME_HEADER);
Status = Fvb->Read (Fvb, 0, 0, &FvhLength, (UINT8 *)&TempFvh);
+ if (EFI_ERROR (Status)) {
+ return Status;
+ }
//
// Allocate a buffer for the caller
@@ -494,9 +497,9 @@ NotifyFwVolBlock ( /**
- This routine is the driver initialization entry point. It initializes the
- libraries, and registers two notification functions. These notification
- functions are responsible for building the FV stack dynamically.
+ This routine is the driver initialization entry point. It registers
+ a notification function. This notification function are responsible
+ for building the FV stack dynamically.
@param ImageHandle The image handle.
@param SystemTable The system table.
|