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

Kaitlin Rupert kaitlin at linux.vnet.ibm.com
Fri Jul 18 23:22:47 UTC 2008


> +
> +        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 at linux.vnet.ibm.com




More information about the Libvirt-cim mailing list