Re: [PATCH 04/16] hyperv: implement connectGetVersion

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2016-08-09 at 12:39 +0000, Jason Miesionczek wrote:
> ---
>  src/hyperv/hyperv_driver.c | 52
> ++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 52 insertions(+)
> 
> diff --git a/src/hyperv/hyperv_driver.c b/src/hyperv/hyperv_driver.c
> index 4a5e80d..b2d1abf 100644
> --- a/src/hyperv/hyperv_driver.c
> +++ b/src/hyperv/hyperv_driver.c
> @@ -1349,6 +1349,57 @@ hypervConnectGetCapabilities(virConnectPtr
> conn)
>      return xml;
>  }
>  
> +static int
> +hypervConnectGetVersion(virConnectPtr conn, unsigned long *version)
> +{
> +    int result = -1;
> +    hypervPrivate *priv = conn->privateData;
> +    CIM_DataFile  *datafile = NULL;
> +    virBuffer query = VIR_BUFFER_INITIALIZER;
> +    char *p;
> +
> +    virBufferAddLit(&query, " Select * from CIM_DataFile where
> Name='c:\\\\windows\\\\system32\\\\vmms.exe' ");

This hard-coded path might not be correct on some systems. I'd pull ir
from Win32_OpereratingSystem->WindowsDirectory:

https://msdn.microsoft.com/en-us/library/aa394239(v=vs.85).aspx


> +    if (hypervGetCIMDataFileList(priv, &query, &datafile) < 0) {
> +        goto cleanup;
> +    }
> +
> +    /* Check the result of convertion */
> +    if (datafile == NULL) {
> +        virReportError(VIR_ERR_INTERNAL_ERROR,
> +                       _("Could not lookup %s for domain %s"),
> +                       "Msvm_VirtualSystemSettingData",

Do you mean CIM_DataFile here?

> +                       datafile->data->Version);

If datafile is NULL then you can't print version like that...

> +        goto cleanup;
> +    }
> +
> +    /* Delete release number and last digit of build number
> 1.1.111x.xxxx */
> +    p = strrchr(datafile->data->Version,'.');
> +    if (p == NULL) {
> +        virReportError(VIR_ERR_INTERNAL_ERROR,
> +                       _("Could not parse version number from
> '%s'"),
> +                       datafile->data->Version);
> +        goto cleanup;
> +    }
> +    p--;
> +    *p = '\0';
> +
> +    /* Parse Version String to Long */
> +    if (virParseVersionString(datafile->data->Version,
> +                              version, true) < 0) {
> +        virReportError(VIR_ERR_INTERNAL_ERROR,
> +                       _("Could not parse version number from
> '%s'"),
> +                       datafile->data->Version);
> +        goto cleanup;
> +    }
> +
> +    result = 0;
> +
> + cleanup:
> +    hypervFreeObject(priv, (hypervObject *)datafile);
> +    virBufferFreeAndReset(&query);
> +
> +    return result;
> +}
>  
>  static virHypervisorDriver hypervHypervisorDriver = {
>      .name = "Hyper-V",
> @@ -1385,6 +1436,7 @@ static virHypervisorDriver
> hypervHypervisorDriver = {
>      .domainManagedSaveRemove = hypervDomainManagedSaveRemove, /*
> 0.9.5 */
>      .connectIsAlive = hypervConnectIsAlive, /* 0.9.8 */
>      .connectGetCapabilities = hypervConnectGetCapabilities, /*
> 1.2.10 */
> +    .connectGetVersion = hypervConnectGetVersion, /* 1.2.10 */
>  };
>  
>  /* Retrieves host system UUID  */

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]