From 62046c1267a14920c629e7b3e0302e5563db79c6 Mon Sep 17 00:00:00 2001
From: Peter Krempa <pkrempa@redhat.com>
Date: Thu, 7 Aug 2014 17:55:12 +0200
Subject: [PATCH] qemu: shared: Split out shared device list remove code

Split it out into a separate function and simplify the code. There's no
need to copy the entry to update it as the hash returns pointer to the
existing item.

Also remove the now unused qemuSharedDeviceEntryCopy function.
---
 src/qemu/qemu_conf.c | 77 +++++++++++++++-----------------------------
 1 file changed, 26 insertions(+), 51 deletions(-)

diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c
index 377d470f8a..61bdb418ef 100644
--- a/src/qemu/qemu_conf.c
+++ b/src/qemu/qemu_conf.c
@@ -924,31 +924,6 @@ qemuSharedDeviceEntryFree(void *payload, const void *name ATTRIBUTE_UNUSED)
     VIR_FREE(entry);
 }
 
-static qemuSharedDeviceEntryPtr
-qemuSharedDeviceEntryCopy(const qemuSharedDeviceEntry *entry)
-{
-    qemuSharedDeviceEntryPtr ret = NULL;
-    size_t i;
-
-    if (VIR_ALLOC(ret) < 0)
-        return NULL;
-
-    if (VIR_ALLOC_N(ret->domains, entry->ref) < 0)
-        goto cleanup;
-
-    for (i = 0; i < entry->ref; i++) {
-        if (VIR_STRDUP(ret->domains[i], entry->domains[i]) < 0)
-            goto cleanup;
-        ret->ref++;
-    }
-
-    return ret;
-
- cleanup:
-    qemuSharedDeviceEntryFree(ret, NULL);
-    return NULL;
-}
-
 
 static int
 qemuSharedDeviceEntryInsert(virQEMUDriverPtr driver,
@@ -1068,6 +1043,31 @@ qemuAddSharedDevice(virQEMUDriverPtr driver,
     return ret;
 }
 
+
+static int
+qemuSharedDeviceEntryRemove(virQEMUDriverPtr driver,
+                            const char *key,
+                            const char *name)
+{
+    qemuSharedDeviceEntryPtr entry = NULL;
+    int idx;
+
+    if (!(entry = virHashLookup(driver->sharedDevices, key)))
+        return -1;
+
+    /* Nothing to do if the shared disk is not recored in the table. */
+    if (!qemuSharedDeviceEntryDomainExists(entry, name, &idx))
+        return 0;
+
+    if (entry->ref != 1)
+        VIR_DELETE_ELEMENT(entry->domains, idx, entry->ref);
+    else
+        ignore_value(virHashRemoveEntry(driver->sharedDevices, key));
+
+    return 0;
+}
+
+
 /* qemuRemoveSharedDevice:
  * @driver: Pointer to qemu driver struct
  * @device: The device def
@@ -1082,15 +1082,12 @@ qemuRemoveSharedDevice(virQEMUDriverPtr driver,
                        virDomainDeviceDefPtr dev,
                        const char *name)
 {
-    qemuSharedDeviceEntryPtr entry = NULL;
-    qemuSharedDeviceEntryPtr new_entry = NULL;
     virDomainDiskDefPtr disk = NULL;
     virDomainHostdevDefPtr hostdev = NULL;
     char *key = NULL;
     char *dev_name = NULL;
     char *dev_path = NULL;
     int ret = -1;
-    int idx;
 
     if (dev->type == VIR_DOMAIN_DEVICE_DISK) {
         disk = dev->data.disk;
@@ -1130,31 +1127,9 @@ qemuRemoveSharedDevice(virQEMUDriverPtr driver,
             goto cleanup;
     }
 
-    if (!(entry = virHashLookup(driver->sharedDevices, key)))
+    if (qemuSharedDeviceEntryRemove(driver, key, name) < 0)
         goto cleanup;
 
-    /* Nothing to do if the shared disk is not recored in
-     * the table.
-     */
-    if (!qemuSharedDeviceEntryDomainExists(entry, name, &idx)) {
-        ret = 0;
-        goto cleanup;
-    }
-
-    if (entry->ref != 1) {
-        if (!(new_entry = qemuSharedDeviceEntryCopy(entry)))
-            goto cleanup;
-
-        VIR_DELETE_ELEMENT(new_entry->domains, idx, new_entry->ref);
-        if (virHashUpdateEntry(driver->sharedDevices, key, new_entry) < 0){
-            qemuSharedDeviceEntryFree(new_entry, NULL);
-            goto cleanup;
-        }
-    } else {
-        if (virHashRemoveEntry(driver->sharedDevices, key) < 0)
-            goto cleanup;
-    }
-
     ret = 0;
  cleanup:
     qemuDriverUnlock(driver);