glusterfs-quota.git - [no description] index : glusterfs-quota.git master origin/upstream_on_quota quota quota-improvements test upstream_on_quota [no description] summaryrefslogtreecommitdiffstats log msg author committer range path: root/glusterfsd diff options context: 12345678910152025303540 space: includeignore mode: unifiedssdiffstat only author Raghavendra Bhat <raghavendra@redhat.com> 2012-11-22 18:48:40 +0530 committer Anand Avati <avati@redhat.com> 2012-12-04 16:28:03 -0800 commit ee2813f440e9b6874eed93de307b8070c8953d5b (patch) tree 7e8f87484af2f954b1c24f23eee899e3a622b054 /glusterfsd parent 6d1607becc4fd84f7422730f37582164c86127d2 (diff) rpc: check the ctx->listener before accessing rpcsvc object Change-Id: I3c913d1fab07deb41eec05c5adc29a3964315b43 BUG: 858487 Signed-off-by: Raghavendra Bhat <raghavendra@redhat.com> Reviewed-on: http://review.gluster.org/4230 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@redhat.com> Diffstat (limited to 'glusterfsd') -rw-r--r-- glusterfsd/src/glusterfsd.c 14 1 files changed, 11 insertions, 3 deletions diff --git a/glusterfsd/src/glusterfsd.c b/glusterfsd/src/glusterfsd.c index 1d28eb8d..2ad391b2 100644 --- a/glusterfsd/src/glusterfsd.c +++ b/glusterfsd/src/glusterfsd.c @@ -973,9 +973,17 @@ cleanup_and_exit (int signum) ctx->cleanup_started = 1; glusterfs_mgmt_pmap_signout (ctx); - if (ctx->listener) { - (void) glusterfs_listener_stop (ctx); - } + + /* below part is a racy code where the rpcsvc object is freed. + * But in another thread (epoll thread), upon poll error in the + * socket the transports are cleaned up where again rpcsvc object + * is accessed (which is already freed by the below function). + * Since the process is about to be killed dont execute the function + * below. + */ + /* if (ctx->listener) { */ + /* (void) glusterfs_listener_stop (ctx); */ + /* } */ /* Call fini() of FUSE xlator first: * so there are no more requests coming and generated by cgit (git 2.34.1) at 2022-09-24 15:56:14 +0000