summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRyan Ding <ryan.ding@open-fs.com>2016-08-23 17:28:44 +0800
committerJeff Darcy <jdarcy@redhat.com>2017-01-30 15:18:18 -0500
commit7f7d7a939e46b330a084d974451eee4757ba61b4 (patch)
treee8b8247795eb4f19a4c58cb1127aaf565e298e80
parent2e0dceb6165d0a93581284fa2e0d74abe4ee615b (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>
-rw-r--r--xlators/features/changelog/src/changelog-rpc-common.c1
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) {