Re: [Xen-devel] [PATCH v4] libxl: introduce LIBXL_DOMAIN_TYPE_INVALID

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Ian Campbell writes ("Re: [PATCH v4] libxl: introduce 
LIBXL_DOMAIN_TYPE_INVALID"):
> so having arranged to call that function at the right time we can assume
> that type is a sensible value, and indeed setdefault makes this the
> case.

Right.

The other situation where we can get _INVALID is if libxl__domain_type
fails, which it can do.

I think this should be handled by having places which call
libxl__domain_type abandon operation and return an error if the
libxl__domain_type fails.

If this is done, then general variables, parameters, etc. within libxl
which are supposed to contain a libxl_domain_type will never contain
_INVALID.

Ian.

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