Re: [PATCH] libxl: Don't derefence null new_name pointer in libxl_domain_rename()

[Ian Campbell <Ian.Campbell@citrix.com>]

On Mon, 2014-12-01 at 14:27 +0000, Euan Harris wrote:
> libxl__domain_rename() unconditionally dereferences its new_name
> parameter, to check whether it is an empty string.   Add a check to
> avoid a segfault if new_name is null.
> 
> Signed-off-by: Euan Harris <euan.harris@citrix.com>

Acked-by: Ian Campbell <ian.campbell@citrix.com>

I think this is a good fix to have for 4.5, Konrad CCd.

> ---
>  tools/libxl/libxl.c |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
> 
> diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c
> index f84f7c2..6e84b5d 100644
> --- a/tools/libxl/libxl.c
> +++ b/tools/libxl/libxl.c
> @@ -385,6 +385,13 @@ int libxl__domain_rename(libxl__gc *gc, uint32_t domid,
>          }
>      }
>  
> +    if (!new_name) {
> +        LIBXL__LOG(ctx, LIBXL__LOG_ERROR,
> +                        "new domain name not specified");
> +        rc = ERROR_INVAL;
> +        goto x_rc;
> +    }
> +
>      if (new_name[0]) {
>          /* nonempty names must be unique */
>          uint32_t domid_e;