]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/libvirt.git/commitdiff
conf: use disk source accessors in locking/
authorEric Blake <eblake@redhat.com>
Mon, 17 Mar 2014 20:23:08 +0000 (14:23 -0600)
committerEric Blake <eblake@redhat.com>
Mon, 24 Mar 2014 17:59:50 +0000 (11:59 -0600)
Part of a series of cleanups to use new accessor methods.

* src/locking/domain_lock.c (virDomainLockManagerAddDisk): Use
accessors.

Signed-off-by: Eric Blake <eblake@redhat.com>
src/locking/domain_lock.c

index 643a8752262c9dc3e222da5b203e94159c0f40fc..ed37dd9f8a6903547e78a9525d98bb40f86327ff 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * domain_lock.c: Locking for domain lifecycle operations
  *
- * Copyright (C) 2010-2011 Red Hat, Inc.
+ * Copyright (C) 2010-2014 Red Hat, Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ -72,12 +72,15 @@ static int virDomainLockManagerAddDisk(virLockManagerPtr lock,
                                        virDomainDiskDefPtr disk)
 {
     unsigned int diskFlags = 0;
-    if (!disk->src)
+    const char *src = virDomainDiskGetSource(disk);
+    int type = virDomainDiskGetType(disk);
+
+    if (!src)
         return 0;
 
-    if (!(disk->type == VIR_DOMAIN_DISK_TYPE_BLOCK ||
-          disk->type == VIR_DOMAIN_DISK_TYPE_FILE ||
-          disk->type == VIR_DOMAIN_DISK_TYPE_DIR))
+    if (!(type == VIR_DOMAIN_DISK_TYPE_BLOCK ||
+          type == VIR_DOMAIN_DISK_TYPE_FILE ||
+          type == VIR_DOMAIN_DISK_TYPE_DIR))
         return 0;
 
     if (disk->readonly)
@@ -85,14 +88,14 @@ static int virDomainLockManagerAddDisk(virLockManagerPtr lock,
     if (disk->shared)
         diskFlags |= VIR_LOCK_MANAGER_RESOURCE_SHARED;
 
-    VIR_DEBUG("Add disk %s", disk->src);
+    VIR_DEBUG("Add disk %s", src);
     if (virLockManagerAddResource(lock,
                                   VIR_LOCK_MANAGER_RESOURCE_TYPE_DISK,
-                                  disk->src,
+                                  src,
                                   0,
                                   NULL,
                                   diskFlags) < 0) {
-        VIR_DEBUG("Failed add disk %s", disk->src);
+        VIR_DEBUG("Failed add disk %s", src);
         return -1;
     }
     return 0;