summaryrefslogtreecommitdiffstats
path: root/xlators/cluster/dht/src/dht-selfheal.c
diff options
context:
space:
mode:
authorAtin Mukherjee <amukherj@redhat.com>2016-02-11 15:37:08 +0530
committerKaushal M <kaushal@redhat.com>2016-08-15 23:27:23 -0700
commit42b5b9bc63891a3447db56224713c5b1028549c5 (patch)
tree5d7b2ccf595a9add3d7438d05cc0124f36deca30 /xlators/cluster/dht/src/dht-selfheal.c
parentcfba6098a4ef8c672969d69678fcdca7c8bb6399 (diff)
rpc : build_prog_details should iterate program list inside critical section
Backport of http://review.gluster.org/13428 While I was analyzing a glusterd crash from free_prog_details, a code walkthrough detects that we iterate over the rpc svc program list without been inside the criticial section. This opens up a possibility of a crash when there is a concurrent writer updating the same list. Solution is to read the list inside lock. > Reviewed-on: http://review.gluster.org/13428 > Smoke: Gluster Build System <jenkins@build.gluster.com> > CentOS-regression: Gluster Build System <jenkins@build.gluster.com> > NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> > Reviewed-by: Niels de Vos <ndevos@redhat.com> > Reviewed-by: Raghavendra G <rgowdapp@redhat.com> BUG: 1310970 Change-Id: Ib4b4b0022a9535e139cd3c00574aab23f07aa9d2 Signed-off-by: Atin Mukherjee <amukherj@redhat.com> Signed-off-by: Oleksandr Natalenko <oleksandr@natalenko.name> Reviewed-on: http://review.gluster.org/13488 Smoke: Gluster Build System <jenkins@build.gluster.org> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Kaushal M <kaushal@redhat.com>
Diffstat (limited to 'xlators/cluster/dht/src/dht-selfheal.c')
0 files changed, 0 insertions, 0 deletions