diff options
| author | Olivier Martin <olivier.martin@arm.com> | 2013-01-10 11:37:12 +0000 |
|---|---|---|
| committer | Steven Kinney <steven.kinney@amd.com> | 2014-04-02 14:46:04 -0500 |
| commit | 31cda3bd5996e4e19fcd3fa75300e59074ae3e8b (patch) | |
| tree | 48ce74ac18f69b590809f5a17a5d227fac3738c0 | |
| parent | e3ff137e3652a85944ba1ba91ad61f09ef1ff248 (diff) | |
MdeModulePkg/MnpDxe: Checked returned value of Snp->GetStatus()
... as explicitly said by the comment.
Signed-off-by: Olivier Martin <olivier.martin@arm.com>
Change-Id: Id9fd51dd5510d6acd04fe2c323a901248c4b85c3
| -rw-r--r-- | MdeModulePkg/Universal/Network/MnpDxe/MnpMain.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/MdeModulePkg/Universal/Network/MnpDxe/MnpMain.c b/MdeModulePkg/Universal/Network/MnpDxe/MnpMain.c index 4c0f3ddd9..9cdbb432e 100644 --- a/MdeModulePkg/Universal/Network/MnpDxe/MnpMain.c +++ b/MdeModulePkg/Universal/Network/MnpDxe/MnpMain.c @@ -79,8 +79,10 @@ MnpGetModeData ( // Upon successful return of GetStatus(), the Snp->Mode->MediaPresent
// will be updated to reflect any change of media status
//
- Snp->GetStatus (Snp, &InterruptStatus, NULL);
- CopyMem (SnpModeData, Snp->Mode, sizeof (*SnpModeData));
+ Status = Snp->GetStatus (Snp, &InterruptStatus, NULL); + if (!EFI_ERROR (Status)) { + CopyMem (SnpModeData, Snp->Mode, sizeof (*SnpModeData)); + } }
if (!Instance->Configured) {
|