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

[Justin Clift]

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>

Hi guys,

This commit is causing a segfault here.  virConnectNumOfStoragePools() 
calls phypNumOfStoragePools(), which calls phypExec(). In this case with 
session equaling NULL.

That's flowing through to libssh2_channel_open_ex(), which tries to 
access data in the (NULL) session structure and segfaults.

Unsure if this mailing list accepts attachments.  I'll try sending 4 
screenshots, that illustrates the problem session variable at each point 
in the backtrace.

(Note the "Target request failed: Cannot access memory at address 
0x4700." in the bottom right of the attached screenshot0001.png file, 
when it attempts to access the session->open_state value.)

Rolling git back 2 commits to this entry, and the above segfault doesn't 
occur:

   8914453 phyp: reduce scope of driver functions

Hope that helps.

Regards and best wishes,

Justin Clift

-- 
Salasaga  -  Open Source eLearning IDE
               http://www.salasaga.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: screenshot0001.png
Type: image/png
Size: 279918 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100627/120a5ff5/attachment-0004.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: screenshot0002.png
Type: image/png
Size: 222495 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100627/120a5ff5/attachment-0005.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: screenshot0003.png
Type: image/png
Size: 217099 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100627/120a5ff5/attachment-0006.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: screenshot0004.png
Type: image/png
Size: 219317 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100627/120a5ff5/attachment-0007.png>