[libvirt] [PATCH 3/8] secret: Rename variable in virSecretObjListAdd

John Ferlan jferlan at redhat.com
Sat Jun 3 13:27:32 UTC 2017


Rename @def to @objdef - it'll make future patches easier.

Signed-off-by: John Ferlan <jferlan at redhat.com>
---
 src/conf/virsecretobj.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/conf/virsecretobj.c b/src/conf/virsecretobj.c
index 62b86b7..e3bcbe5 100644
--- a/src/conf/virsecretobj.c
+++ b/src/conf/virsecretobj.c
@@ -332,7 +332,7 @@ virSecretObjListAdd(virSecretObjListPtr secrets,
                     virSecretDefPtr *oldDef)
 {
     virSecretObjPtr obj;
-    virSecretDefPtr def;
+    virSecretDefPtr objdef;
     virSecretObjPtr ret = NULL;
     char uuidstr[VIR_UUID_STRING_BUFLEN];
     char *configFile = NULL, *base64File = NULL;
@@ -347,26 +347,26 @@ virSecretObjListAdd(virSecretObjListPtr secrets,
     /* Is there a secret already matching this UUID */
     if ((obj = virSecretObjListFindByUUIDLocked(secrets, uuidstr))) {
         virObjectLock(obj);
-        def = obj->def;
+        objdef = obj->def;
 
-        if (STRNEQ_NULLABLE(def->usage_id, newdef->usage_id)) {
+        if (STRNEQ_NULLABLE(objdef->usage_id, newdef->usage_id)) {
             virReportError(VIR_ERR_INTERNAL_ERROR,
                            _("a secret with UUID %s is already defined for "
                              "use with %s"),
-                           uuidstr, def->usage_id);
+                           uuidstr, objdef->usage_id);
             goto cleanup;
         }
 
-        if (def->isprivate && !newdef->isprivate) {
+        if (objdef->isprivate && !newdef->isprivate) {
             virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                            _("cannot change private flag on existing secret"));
             goto cleanup;
         }
 
         if (oldDef)
-            *oldDef = def;
+            *oldDef = objdef;
         else
-            virSecretDefFree(def);
+            virSecretDefFree(objdef);
         obj->def = newdef;
     } else {
         /* No existing secret with same UUID,
@@ -375,8 +375,8 @@ virSecretObjListAdd(virSecretObjListPtr secrets,
                                                      newdef->usage_type,
                                                      newdef->usage_id))) {
             virObjectLock(obj);
-            def = obj->def;
-            virUUIDFormat(def->uuid, uuidstr);
+            objdef = obj->def;
+            virUUIDFormat(objdef->uuid, uuidstr);
             virReportError(VIR_ERR_INTERNAL_ERROR,
                            _("a secret with UUID %s already defined for "
                              "use with %s"),
-- 
2.9.4




More information about the libvir-list mailing list