]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
storage: Use virStoragePoolObj{Is|Set}Autostart
authorJohn Ferlan <jferlan@redhat.com>
Wed, 26 Jul 2017 16:23:11 +0000 (12:23 -0400)
committerJohn Ferlan <jferlan@redhat.com>
Tue, 19 Sep 2017 12:30:19 +0000 (08:30 -0400)
Use the new accessor APIs for storage_driver and test_driver.

src/storage/storage_driver.c
src/test/test_driver.c

index 37e9981b5bf0fbbd551ebe68faaac68dcf9b5364..4b7bad82df95dfbf5fb27c1f5bf6a34b0b89530a 100644 (file)
@@ -196,7 +196,7 @@ storageDriverAutostart(void)
             continue;
         }
 
-        if (obj->autostart &&
+        if (virStoragePoolObjIsAutostart(obj) &&
             !virStoragePoolObjIsActive(obj)) {
             if (backend->startPool &&
                 backend->startPool(conn, obj) < 0) {
@@ -1249,11 +1249,7 @@ storagePoolGetAutostart(virStoragePoolPtr pool,
     if (virStoragePoolGetAutostartEnsureACL(pool->conn, obj->def) < 0)
         goto cleanup;
 
-    if (!virStoragePoolObjGetConfigFile(obj)) {
-        *autostart = 0;
-    } else {
-        *autostart = obj->autostart;
-    }
+    *autostart = virStoragePoolObjIsAutostart(obj) ? 1 : 0;
 
     ret = 0;
 
@@ -1269,6 +1265,8 @@ storagePoolSetAutostart(virStoragePoolPtr pool,
     virStoragePoolObjPtr obj;
     const char *configFile;
     const char *autostartLink;
+    bool new_autostart;
+    bool cur_autostart;
     int ret = -1;
 
     storageDriverLock();
@@ -1286,10 +1284,10 @@ storagePoolSetAutostart(virStoragePoolPtr pool,
 
     autostartLink = virStoragePoolObjGetAutostartLink(obj);
 
-    autostart = (autostart != 0);
-
-    if (obj->autostart != autostart) {
-        if (autostart) {
+    new_autostart = (autostart != 0);
+    cur_autostart = virStoragePoolObjIsAutostart(obj);
+    if (cur_autostart != new_autostart) {
+        if (new_autostart) {
             if (virFileMakePath(driver->autostartDir) < 0) {
                 virReportSystemError(errno,
                                      _("cannot create autostart directory %s"),
@@ -1312,7 +1310,7 @@ storagePoolSetAutostart(virStoragePoolPtr pool,
                 goto cleanup;
             }
         }
-        obj->autostart = autostart;
+        virStoragePoolObjSetAutostart(obj, autostart);
     }
 
     ret = 0;
index d8ae2dea436bcfb918c5fa04faa156b35dd2bad9..e28ad6d964751902c7b1943c2a717f56055b0a93 100644 (file)
@@ -4764,7 +4764,7 @@ testStoragePoolGetAutostart(virStoragePoolPtr pool,
     if (!virStoragePoolObjGetConfigFile(obj))
         *autostart = 0;
     else
-        *autostart = obj->autostart;
+        *autostart = virStoragePoolObjIsAutostart(obj) ? 1 : 0;
 
     virStoragePoolObjUnlock(obj);
     return 0;
@@ -4789,7 +4789,7 @@ testStoragePoolSetAutostart(virStoragePoolPtr pool,
     }
 
     autostart = (autostart != 0);
-    obj->autostart = autostart;
+    virStoragePoolObjSetAutostart(obj, autostart);
     ret = 0;
 
  cleanup: