Bug 1212684

Summary:	GlusterD segfaults when started with management SSL
Product:	[Community] GlusterFS	Reporter:	Kaushal <kaushal>
Component:	glusterd	Assignee:	Kaushal <kaushal>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	3.6.2	CC:	bugs, gluster-bugs, rabhat
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-v3.6.7	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-12-04 08:43:08 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1184460

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>