Hi all, this patch contains two small fixes: - set the domid of the guest and not the one of the stubdom in the libxl_device_model_starting returned to the user; - check that the length of the two strings matches in libxl_name_to_domid, otherwise we can get a match for two different domains that have the same initial part of the name. Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> --- diff -r 0abcf45319ca tools/libxl/libxl.c --- a/tools/libxl/libxl.c Tue Dec 01 15:58:16 2009 +0000 +++ b/tools/libxl/libxl.c Tue Dec 01 15:58:43 2009 +0000 @@ -842,7 +842,7 @@ if (starting_r) { *starting_r = libxl_calloc(ctx, sizeof(libxl_device_model_starting), 1); - (*starting_r)->domid = domid; + (*starting_r)->domid = info->domid; (*starting_r)->dom_path = libxl_xs_get_dompath(ctx, info->domid); (*starting_r)->for_spawn = NULL; } diff -r 0abcf45319ca tools/libxl/libxl_utils.c --- a/tools/libxl/libxl_utils.c Tue Dec 01 15:58:16 2009 +0000 +++ b/tools/libxl/libxl_utils.c Tue Dec 01 15:58:43 2009 +0000 @@ -68,7 +68,7 @@ for (i = 0; i < num; i++) { snprintf(path, sizeof(path), "/local/domain/%s/name", l[i]); domname = xs_read(ctx->xsh, XBT_NULL, path, &len); - if (domname != NULL && !strncmp(domname, name, len)) { + if (domname != NULL && len == strlen(name) && !strncmp(domname, name, len)) { int domid_i = atoi(l[i]); for (j = 0; j < nb_domains; j++) { if (dominfo[j].domid == domid_i) { _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel