]> xenbits.xensource.com Git - libvirt.git/commitdiff
storageBackendCreateQemuImgSecretPath: Use virSecureErase instead of VIR_DISPOSE_N
authorPeter Krempa <pkrempa@redhat.com>
Mon, 1 Feb 2021 13:13:53 +0000 (14:13 +0100)
committerPeter Krempa <pkrempa@redhat.com>
Wed, 3 Feb 2021 12:07:13 +0000 (13:07 +0100)
Clear out the value using virSecureErase and free it with g_free so
that VIR_DISPOSE_N can be phased out.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
src/storage/storage_util.c

index 3d8de163415d1df98254430f01cc967663bd2a48..b5adb05826105ab59f0eea88fe95e3d135adcffa 100644 (file)
@@ -74,6 +74,7 @@
 #include "virxml.h"
 #include "virfdstream.h"
 #include "virutil.h"
+#include "virsecureerase.h"
 
 #define VIR_FROM_THIS VIR_FROM_STORAGE
 
@@ -1314,7 +1315,8 @@ storageBackendCreateQemuImgSecretPath(virStoragePoolObjPtr pool,
 
  cleanup:
     virObjectUnref(conn);
-    VIR_DISPOSE_N(secret, secretlen);
+    virSecureErase(secret, secretlen);
+    g_free(secret);
 
     return secretPath;