On Tue, Apr 11, 2017 at 05:53:50PM +0200, Peter Krempa wrote:
Similarly to virshDomainFree add a wrapper for the snapshot object
freeing function.
---
tools/virsh-snapshot.c | 48 ++++++++++++++++++------------------------------
tools/virsh-util.c | 10 ++++++++++
tools/virsh-util.h | 3 +++
3 files changed, 31 insertions(+), 30 deletions(-)
diff --git a/tools/virsh-util.c b/tools/virsh-util.c
index 5fdb40855..e225d3332 100644
--- a/tools/virsh-util.c
+++ b/tools/virsh-util.c
@@ -160,3 +160,13 @@ virshDomainFree(virDomainPtr dom)
virDomainFree(dom); /* sc_prohibit_obj_free_apis_in_virsh */
}
+
+
+void
+virshDomainSnapshotFree(virDomainSnapshotPtr snap)
+{
+ if (!snap)
+ return;
+
+ virDomainSnapshotFree(snap); /* sc_prohibit_obj_free_apis_in_virsh */
+}
This adds an exemption from the syntax-check without actually enhancing
the check.
ACK if you tweak the regex in sc_prohibit_obj_free_apis_in_virsh
Jan