summaryrefslogtreecommitdiff
path: root/OvmfPkg
diff options
context:
space:
mode:
authorThomas Huth <thuth@redhat.com>2017-01-19 10:17:37 +0100
committerLaszlo Ersek <lersek@redhat.com>2017-01-19 10:28:40 +0100
commit19ca06bb84cafd661f7da34e9ca3c7ec6add1135 (patch)
tree1ad0d32c29b918f1a7e9026f8f6af607529597f4 /OvmfPkg
parent266746561d454d3d35938886e61a02b1ee1abaff (diff)
downloadedk2-19ca06bb84cafd661f7da34e9ca3c7ec6add1135.zip
edk2-19ca06bb84cafd661f7da34e9ca3c7ec6add1135.tar.gz
edk2-19ca06bb84cafd661f7da34e9ca3c7ec6add1135.tar.bz2
OvmfPkg: Remove superfluous return statements.
If the code eventually returns "Status" anyway, it does not make sense to explicitely return "Status" in case of an error, too. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Thomas Huth <thuth@redhat.com> Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Diffstat (limited to 'OvmfPkg')
-rw-r--r--OvmfPkg/Library/NvVarsFileLib/FsAccess.c3
-rw-r--r--OvmfPkg/XenBusDxe/XenStore.c3
2 files changed, 0 insertions, 6 deletions
diff --git a/OvmfPkg/Library/NvVarsFileLib/FsAccess.c b/OvmfPkg/Library/NvVarsFileLib/FsAccess.c
index 826901f..cd0bfff 100644
--- a/OvmfPkg/Library/NvVarsFileLib/FsAccess.c
+++ b/OvmfPkg/Library/NvVarsFileLib/FsAccess.c
@@ -77,9 +77,6 @@ GetNvVarsFile (
),
0
);
- if (EFI_ERROR (Status)) {
- return Status;
- }
return Status;
}
diff --git a/OvmfPkg/XenBusDxe/XenStore.c b/OvmfPkg/XenBusDxe/XenStore.c
index 1666c4b..76b6e29 100644
--- a/OvmfPkg/XenBusDxe/XenStore.c
+++ b/OvmfPkg/XenBusDxe/XenStore.c
@@ -1075,9 +1075,6 @@ XenStoreInit (
/* Initialize the shared memory rings to talk to xenstored */
Status = XenStoreInitComms (&xs);
- if (EFI_ERROR (Status)) {
- return Status;
- }
return Status;
}