Re: [Xen-devel] [PATCH] libxl: set rc to 0 in init_acpi_config in success path

[Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>]

On 12/16/2016 10:51 AM, Wei Liu wrote:
> xc_doamin_getinfo returns >=0 in success path, and if there is no vnode
> configured, that rc will be returned to caller, which indicates error.
>
> Fix that by setting rc to 0 in success path.
>
> Reported-by: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>
> Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
> ---
> Cc: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>
> Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
>
> Please squash this with "libxl: init_acpi_config should return rc
> in exit path" when backporting to 4.8.
> ---
>  tools/libxl/libxl_x86_acpi.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/tools/libxl/libxl_x86_acpi.c b/tools/libxl/libxl_x86_acpi.c
> index 6cf0c30..173188f 100644
> --- a/tools/libxl/libxl_x86_acpi.c
> +++ b/tools/libxl/libxl_x86_acpi.c
> @@ -153,6 +153,7 @@ static int init_acpi_config(libxl__gc *gc,
>      config->lapic_id = acpi_lapic_id;
>      config->acpi_revision = 5;
>  
> +    rc = 0;
>  out:
>      return rc;
>  }

You can also drop 'int rc = 0;' definition.

Either way,

Tested-by: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel