diff options
author | Xavier Hernandez <xhernandez@datalab.es> | 2016-06-09 16:53:19 +0200 |
---|---|---|
committer | Raghavendra G <rgowdapp@redhat.com> | 2016-06-15 02:04:29 -0700 |
commit | 4c08d36e7c6f189499f2340eb529b7f4ceff57f6 (patch) | |
tree | 825d74e65a7bfad50bace21df80cfb2545083814 /geo-replication/syncdaemon/syncdutils.py | |
parent | e740b700975bdb1014ede99c65731095bdf81a72 (diff) |
cluster/dht: Fix unsafe iteration on inode->fd_list
When DHT traverses the inode->fd_list, it does that in an unsafe
way that can generate races with fd_unref() called from other threads.
This patch fixes this problem taking the inode->lock and adding a
reference to the fd while it's being used outside of the mutex
protected region.
A minor change in storage/posix has been done to also access the
inode->fd_list in a safe way.
Change-Id: I10d469ca6a8f76e950a8c9779ae9c8b70f88ef93
BUG: 1344340
Signed-off-by: Xavier Hernandez <xhernandez@datalab.es>
Reviewed-on: http://review.gluster.org/14682
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
Diffstat (limited to 'geo-replication/syncdaemon/syncdutils.py')
0 files changed, 0 insertions, 0 deletions