libxl: libxl_device_console.build_state is const
authorIan Campbell <ian.campbell@citrix.com>
Thu, 19 Aug 2010 14:21:01 +0000 (15:21 +0100)
committerIan Campbell <ian.campbell@citrix.com>
Thu, 19 Aug 2010 14:21:01 +0000 (15:21 +0100)
This field always refers to a libxl_domain_build_state structure which
is managed already by the caller, therefore it is incorrect to free it
as part of libxl_device_console_destroy.

(arguably this field should be a copy of, not a reference too, the original)

[PATCH 06 of 16 of
 libxl: autogenerate type definitions and destructor functions]

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
tools/libxl/libxl.idl

index ea425de6b3c2248bd0ad052545c38474a5edd845..e3cca5a8def0934603cf9ddf1bc819da3322653a 100644 (file)
@@ -195,7 +195,7 @@ libxl_device_console = Struct("device_console", [
     ("domid", domid),
     ("devid", integer),
     ("consback", libxl_console_consback),
-    ("build_state", Reference(libxl_domain_build_state)),
+    ("build_state", Reference(libxl_domain_build_state), True),
     ("output", string),
     ])