[libvirt PATCH 32/39] cpu_x86: Introduce virCPUx86SignaturesFree

Jiri Denemark posted 39 patches 5 years, 10 months ago
[libvirt PATCH 32/39] cpu_x86: Introduce virCPUx86SignaturesFree
Posted by Jiri Denemark 5 years, 10 months ago
The function will be used for freeing virCPUx86Signatures structure
introduced later in this series.

Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
---
 src/cpu/cpu_x86.c | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/src/cpu/cpu_x86.c b/src/cpu/cpu_x86.c
index 182d0da97c..a71b5ccacc 100644
--- a/src/cpu/cpu_x86.c
+++ b/src/cpu/cpu_x86.c
@@ -1090,6 +1090,13 @@ x86FeatureParse(xmlXPathContextPtr ctxt,
 }
 
 
+static void
+virCPUx86SignaturesFree(uint32_t *signatures)
+{
+    g_free(signatures);
+}
+
+
 static int
 virCPUx86SignaturesCopy(virCPUx86ModelPtr dst,
                         virCPUx86ModelPtr src)
@@ -1149,7 +1156,7 @@ x86ModelFree(virCPUx86ModelPtr model)
         return;
 
     g_free(model->name);
-    g_free(model->signatures);
+    virCPUx86SignaturesFree(model->signatures);
     virCPUx86DataClear(&model->data);
     g_free(model);
 }
@@ -1414,7 +1421,7 @@ x86ModelParseSignatures(virCPUx86ModelPtr model,
         return n;
 
     /* Remove inherited signatures. */
-    VIR_FREE(model->signatures);
+    virCPUx86SignaturesFree(model->signatures);
 
     model->nsignatures = n;
     if (VIR_ALLOC_N(model->signatures, n) < 0)
-- 
2.26.0

Re: [libvirt PATCH 32/39] cpu_x86: Introduce virCPUx86SignaturesFree
Posted by Ján Tomko 5 years, 10 months ago
On a Friday in 2020, Jiri Denemark wrote:
>The function will be used for freeing virCPUx86Signatures structure
>introduced later in this series.
>
>Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
>---
> src/cpu/cpu_x86.c | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano