From: John Ferlan <jferlan@redhat.com>
Date: Mon, 2 Dec 2013 20:31:42 +0000 (-0500)
Subject: storage_driver: Resolve Coverity CHECKED_RETURN warning
X-Git-Tag: CVE-2013-6436~128
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=37a02bab1e718706aa5af65f34ea2bfee1649854;p=thirdparty%2Flibvirt.git

storage_driver: Resolve Coverity CHECKED_RETURN warning

The storageRegister() didn't check the return from the
virRegisterStorageDriver() like other callers did, so Coverity
flagged it.  Just check the return and handle.
---

diff --git a/src/storage/storage_driver.c b/src/storage/storage_driver.c
index 3b4715a240..f08255ea0e 100644
--- a/src/storage/storage_driver.c
+++ b/src/storage/storage_driver.c
@@ -2644,8 +2644,10 @@ static virStateDriver stateDriver = {
     .stateReload = storageStateReload,
 };
 
-int storageRegister(void) {
-    virRegisterStorageDriver(&storageDriver);
+int storageRegister(void)
+{
+    if (virRegisterStorageDriver(&storageDriver) < 0)
+        return -1;
     virRegisterStateDriver(&stateDriver);
     return 0;
 }