[libvirt] [PATCHv3 3/4] phyp: add storage management driver

[Eric Blake]

On 06/27/2010 09:17 AM, Justin Clift wrote:
> On 06/25/2010 08:32 AM, Eric Blake wrote:
>> From: Eduardo Otubo<otubo at linux.vnet.ibm.com>
>>
>> Add the storage management driver to the Power Hypervisor driver.
>> This is a big but simple patch, it's just a new set of functions.
>>
>> This patch includes:
>>   * Storage driver: The set of pool-* and vol-* functions.
>>   * attach-disk function.
>>   * Support for IVM on the new functions.
>>
>> Signed-off-by: Eric Blake<eblake at redhat.com>
> <snip>
> 
> This commit is causing a segfault here when running "pool-list" in
> unmodified virsh.
> 
> virConnectNumOfStoragePools() calls phypNumOfStoragePools(), which calls
> phypExec(). In this case with session equaling NULL.

virConnectNumOfStoragePools should not call successfully
phypNumOfStoragePools unless the connection URI is for phyp.  Does this
patch clean up the mess?
https://www.redhat.com/archives/libvir-list/2010-June/msg00707.html

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100628/5d9ac302/attachment-0001.sig>