summaryrefslogtreecommitdiffstats
path: root/xlators/mgmt/glusterd/src/glusterd-handler.c
diff options
context:
space:
mode:
authorKaushal M <kaushal@redhat.com>2016-03-01 13:04:03 +0530
committerRaghavendra G <rgowdapp@redhat.com>2016-03-03 20:42:14 -0800
commitd117466422b2fe97390b9ccc7a3c277e7a64285a (patch)
treed80b3554be36e8440f08a0e1f17875d9662e0297 /xlators/mgmt/glusterd/src/glusterd-handler.c
parentfdd8fc6f82e3b47eb47c82335f79552610596913 (diff)
socket: Launch socket_poller only if connect succeeded
For an encrypted connection, sockect_connect() used to launch socket_poller() in it's own thread (ON by default), even if the connect failed. This would cause two unrefs to be done on the transport, once in socket_poller() and once in socket_connect(), causing the transport to be freed and cleaned up. This would cause further reconnect attempts from failing as the transport wouldn't be available. By starting socket_poller() only if connect succeeded, this is avoided. Change-Id: Ie22090dbb1833bdd0f636a76cb3935d766711917 BUG: 1313206 Signed-off-by: Kaushal M <kaushal@redhat.com> Reviewed-on: http://review.gluster.org/13554 Smoke: Gluster Build System <jenkins@build.gluster.com> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-handler.c')
0 files changed, 0 insertions, 0 deletions