Re: [PATCH 4/5] hyperv: Implement virDomainSnapshotCurrent()/virDomainHasCurrentSnapshot()

[Michal Prívozník via Devel]

On 3/6/26 23:12, Jonathon Jongsma via Devel wrote:
> Signed-off-by: Jonathon Jongsma <jjong...@redhat.com>
> ---
>  src/hyperv/hyperv_driver.c | 48 ++++++++++++++++++++++++++++++++++++++
>  1 file changed, 48 insertions(+)
> 
> diff --git a/src/hyperv/hyperv_driver.c b/src/hyperv/hyperv_driver.c
> index 4a4db0e76b..dd1248ced1 100644
> --- a/src/hyperv/hyperv_driver.c
> +++ b/src/hyperv/hyperv_driver.c
> @@ -4235,6 +4235,52 @@ hypervParseInstanceIdFromParentPath(const char 
> *obj_path)
>  }
>  
>  
> +static int
> +hypervDomainHasCurrentSnapshot(virDomainPtr domain,
> +                               unsigned int flags)
> +{
> +    g_autoptr(Msvm_VirtualSystemSettingData) vssd = NULL;
> +    hypervPrivate *priv = domain->conn->privateData;
> +    char uuid_string[VIR_UUID_STRING_BUFLEN];
> +
> +    virCheckFlags(0, -1);
> +
> +    virUUIDFormat(domain->uuid, uuid_string);
> +
> +    if (hypervGetMsvmVirtualSystemSettingDataFromUUID(priv, uuid_string, 
> &vssd) < 0)
> +        return -1;
> +
> +    return (vssd->data->Parent != NULL);

Drop the parentheses.

Michal