[Xen-devel] [PATCH 0/6] libxl, xl: fix domain name uniqueness check

[Ian Jackson <ian.jackson@xxxxxxxxxxxxx>]

We knew that checking for the domain name uniqueness in
parse_config_data in xl was wrong.  Sadly not only is it wrong, but it
doesn't work properly because it prevents the parsing of domain config
files in various other contextx.

Instead, in this series, we put this check in libxl_domain_rename,
which is the right place for it to be.  This function is indeed called
during domain creation to set the domain's name at the start.  As a
sop to those who think domain names are not important, we allow any
number of domains with the name "".  This is not recommended.

However, after moving this check down into the bowels of libxl domain
creation, it became obvious during testing that the error paths with
are taken when the check fails didn't work properly and generated a
lot of noisy output.

So in this series:
  patch 1:     revert the original broken check
  patches 2-5: error path fixes
  patch 6:     add the check to libxl_domain_rename

Signed-off-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel