[libvirt] [libvirt-glib 1/4] gobject: don't try to use pool's volumes before a successful refresh

Zeeshan Ali (Khattak) zeeshanak at gnome.org
Wed Nov 14 13:05:11 UTC 2012


On Wed, Nov 14, 2012 at 10:31 AM, Christophe Fergeau
<cfergeau at redhat.com> wrote:
> On Tue, Nov 13, 2012 at 06:45:56PM +0100, Zeeshan Ali (Khattak) wrote:
>> On Tue, Nov 6, 2012 at 1:45 PM, Christophe Fergeau <cfergeau at redhat.com> wrote:
>> > gvir_storage_pool_refresh must be called and must be successful before
>> > trying to use gvir_storage_pool_get_volume,
>> > gvir_storage_pool_get_volumes and gvir_storage_pool_create_volume.
>> > As the storage pool refresh can fail for reasons external to
>> > libvirt/libvirt-gobject, the library user should check _refresh
>> > errors. This commit outputs runtime warnings when these functions
>> > are called and GVirObjectStoragePool::priv::volumes is NULL.
>>
>> Looks good. ACK!
>
> Just this patch or the whole series?

I've been reviewing/ACK'ing each patch separately so this one is only
for this patch. :) Let me know if I missed a patch.

-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124




More information about the libvir-list mailing list