From b668cbfa2503aeae297f9783a064ffd5dd465658 Mon Sep 17 00:00:00 2001 From: Olivier Martin Date: Thu, 10 Jan 2013 11:37:12 +0000 Subject: MdeModulePkg/MnpDxe: Checked returned value of Snp->GetStatus() ... as explicitly said by the comment. Signed-off-by: Olivier Martin Change-Id: Id9fd51dd5510d6acd04fe2c323a901248c4b85c3 --- 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 4c0f3dd..8940964 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) { -- 1.7.0.4