diff options
| author | Olivier Martin <olivier.martin@arm.com> | 2013-01-10 11:37:12 +0000 |
|---|---|---|
| committer | Leif Lindholm <leif.lindholm@linaro.org> | 2014-01-24 15:44:20 +0000 |
| commit | a7df1d6eab49a20343a7fab06bc859b8668d808c (patch) | |
| tree | 611092816b4b871a408001835b171ada85941649 | |
| parent | 48e2762bcd193e247e5998d64bd713053b43e2ea (diff) | |
| download | juice-a7df1d6eab49a20343a7fab06bc859b8668d808c.tar.gz | |
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) {
|