Re: [Libvirt-cim] [PATCH] Persist RequestedState in infostore

+ + infostore = infostore_open(dom); + if (infostore != NULL) { + infostore_set_u64(infostore, "reqstate", (uint64_t)state); + infostore_close(infostore); + }

Was going to comment that infostore_close() should go outside the if, but I'd forgotten that infostore_open() cleanups before the return if the call fails. I tested this out and it looks good. +1 -- Kaitlin Rupert IBM Linux Technology Center kaitlin@linux.vnet.ibm.com