Bug 1212684 - GlusterD segfaults when started with management SSL
Summary: GlusterD segfaults when started with management SSL
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: glusterd
Version: 3.6.2
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Kaushal
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: glusterfs-3.6.3
TreeView+ depends on / blocked
 
Reported: 2015-04-17 06:01 UTC by Kaushal
Modified: 2015-12-04 08:43 UTC (History)
3 users (show)

Fixed In Version: glusterfs-v3.6.7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-12-04 08:43:08 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Kaushal 2015-04-17 06:01:40 UTC
GlusterD segfaults and crashes on startup when management SSL is enabled. This happens when other peers are down and a socket connection cannot be established to the other peers.

Because of a bug in the RPC layer, the transport tries to do a SSL write even when the connection is not established and SSL is not setup. The write leads to a segfault as the SSL context is not created yet.

Commit 0b9a6a6(socket: fix segfaults when TLS management connections fail) from Jeff Darcy on master fixes this issue, and needs to be backported.

Comment 1 Anand Avati 2015-04-17 06:06:08 UTC
REVIEW: http://review.gluster.org/10280 (socket: fix segfaults when TLS management connections fail) posted (#1) for review on release-3.6 by Kaushal M (kaushal)

Comment 2 Anand Avati 2015-04-21 12:27:17 UTC
COMMIT: http://review.gluster.org/10280 committed in release-3.6 by Raghavendra Bhat (raghavendra) 
------
commit 415f575279d30feccb7436b85121a3a5489fdcf7
Author: Jeff Darcy <jdarcy>
Date:   Wed Nov 5 22:37:48 2014 -0500

    socket: fix segfaults when TLS management connections fail
    
    Backport of 0b9a6a6 from master.
    
    BUG: 1212684
    Change-Id: Iec5410b937af150621b757924836cec638b5eb55
    Signed-off-by: Jeff Darcy <jdarcy>
    Signed-off-by: Kaushal M <kaushal>
    Reviewed-on: http://review.gluster.org/10280
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Raghavendra Bhat <raghavendra>


Note You need to log in before you can comment on or make changes to this bug.