diff options
author | Pranith Kumar K <pranithk@gluster.com> | 2012-04-16 14:42:56 +0530 |
---|---|---|
committer | Anand Avati <avati@redhat.com> | 2012-05-31 16:53:00 -0700 |
commit | 4bc0fa312f8a64c7ee0f52e5635d7f082454aaa2 (patch) | |
tree | 0942b392b9d795419722d3e9119726e36b1cb580 /xlators/mgmt | |
parent | d2ac835ba0d51ce1679dc71640472eecbfbaa704 (diff) |
storage/posix: Prevent gfid handle leaks
The case which can lead to gfid handle leaks:
Self-heal removes directory '/d' with 10 files in it, in brick b1.
This dir is renamed to <landfill>/<hashval of '<brick-path>/d'>
by posix. Before the janitor thread could remove the directory,
self-heal could remove another directory with same path '/d'.
Then again the rename to same path is done by posix as before.
The gfid-handles of the old '/d', 10 files in it are not unlinked.
To prevent such problems, rename the directory to be removed to
<landfill>/<gfid-str>.
Change-Id: Iad13708e1ebcc5222b64c058aa9a2d372e1bfa5b
BUG: 811970
Signed-off-by: Pranith Kumar K <pranithk@gluster.com>
Reviewed-on: http://review.gluster.com/3159
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'xlators/mgmt')
0 files changed, 0 insertions, 0 deletions