diff options
author | Ryan Ding <ryan.ding@open-fs.com> | 2016-08-23 17:28:44 +0800 |
---|---|---|
committer | Jeff Darcy <jdarcy@redhat.com> | 2017-01-30 15:18:18 -0500 |
commit | 7f7d7a939e46b330a084d974451eee4757ba61b4 (patch) | |
tree | e8b8247795eb4f19a4c58cb1127aaf565e298e80 /xlators/features/changelog | |
parent | 2e0dceb6165d0a93581284fa2e0d74abe4ee615b (diff) |
features/changelog: fix a bug in changelog_rpc_server_destroy
the programe unregister loop never get moved forward, it's a dead loop.
to resolve it, add progs++ in the loop.
Change-Id: Ib25ded10b3ce808e2fb57b13d767833f24cf01a7
BUG: 1369393
Signed-off-by: Ryan Ding <ryan.ding@open-fs.com>
Reviewed-on: https://review.gluster.org/15292
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Prashanth Pai <ppai@redhat.com>
Tested-by: Kotresh HR <khiremat@redhat.com>
Reviewed-by: Kotresh HR <khiremat@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Jeff Darcy <jdarcy@redhat.com>
Diffstat (limited to 'xlators/features/changelog')
-rw-r--r-- | xlators/features/changelog/src/changelog-rpc-common.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/xlators/features/changelog/src/changelog-rpc-common.c b/xlators/features/changelog/src/changelog-rpc-common.c index c0744ef6218..08cd41e4f17 100644 --- a/xlators/features/changelog/src/changelog-rpc-common.c +++ b/xlators/features/changelog/src/changelog-rpc-common.c @@ -266,6 +266,7 @@ changelog_rpc_server_destroy (xlator_t *this, rpcsvc_t *rpc, char *sockfile, while (*progs) { prog = *progs; (void) rpcsvc_program_unregister (rpc, prog); + progs++; } list_for_each_entry_safe (listener, next, &rpc->listeners, list) { |