|
 |
887953 |
From 3a1484c401d4293531c80532fa96c2f7cfc8aa2d Mon Sep 17 00:00:00 2001
|
|
|
887953 |
From: Sanju Rakonde <srakonde@redhat.com>
|
|
|
887953 |
Date: Wed, 19 Sep 2018 19:49:51 +0530
|
|
|
887953 |
Subject: [PATCH 458/493] glusterd: fix crash
|
|
|
887953 |
|
|
|
887953 |
When huge number of volumes are created, glusterd crash is seen.
|
|
|
887953 |
With the core dump, got to know that mgmt_lock_timer became NULL.
|
|
|
887953 |
Adding a null check for the same, need to explore about the root
|
|
|
887953 |
cause.
|
|
|
887953 |
|
|
|
887953 |
>updates: bz#1630922
|
|
|
887953 |
>Change-Id: I0770063fcbbbf4b24bef29e94b857b20bdfb5b85
|
|
|
887953 |
>Signed-off-by: Sanju Rakonde <srakonde@redhat.com>
|
|
|
887953 |
|
|
|
887953 |
upstream patch: https://review.gluster.org/#/c/glusterfs/+/21228/
|
|
|
887953 |
|
|
|
887953 |
Change-Id: I0770063fcbbbf4b24bef29e94b857b20bdfb5b85
|
|
|
887953 |
BUG: 1599220
|
|
|
887953 |
Signed-off-by: Sanju Rakonde <srakonde@redhat.com>
|
|
|
887953 |
Reviewed-on: https://code.engineering.redhat.com/gerrit/158542
|
|
|
887953 |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
887953 |
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
|
887953 |
---
|
|
|
887953 |
xlators/mgmt/glusterd/src/glusterd-locks.c | 2 +-
|
|
|
887953 |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
887953 |
|
|
|
887953 |
diff --git a/xlators/mgmt/glusterd/src/glusterd-locks.c b/xlators/mgmt/glusterd/src/glusterd-locks.c
|
|
|
887953 |
index d75452d..d62d9dd 100644
|
|
|
887953 |
--- a/xlators/mgmt/glusterd/src/glusterd-locks.c
|
|
|
887953 |
+++ b/xlators/mgmt/glusterd/src/glusterd-locks.c
|
|
|
887953 |
@@ -890,7 +890,7 @@ glusterd_mgmt_v3_unlock (const char *name, uuid_t uuid, char *type)
|
|
|
887953 |
type, name);
|
|
|
887953 |
|
|
|
887953 |
/* Release owner reference which was held during lock */
|
|
|
887953 |
- if (mgmt_lock_timer->timer) {
|
|
|
887953 |
+ if (mgmt_lock_timer && mgmt_lock_timer->timer) {
|
|
|
887953 |
ret = -1;
|
|
|
887953 |
mgmt_lock_timer_xl = mgmt_lock_timer->xl;
|
|
|
887953 |
GF_VALIDATE_OR_GOTO (this->name, mgmt_lock_timer_xl, out);
|
|
|
887953 |
--
|
|
|
887953 |
1.8.3.1
|
|
|
887953 |
|