diff options
| author | Edward Shishkin <edward@redhat.com> | 2014-12-01 22:15:02 +0100 |
|---|---|---|
| committer | Raghavendra Bhat <raghavendra@redhat.com> | 2014-12-09 22:25:47 -0800 |
| commit | b7d71e6541bc9c4a33cc5b46e93524ca1990c0bf (patch) | |
| tree | 4deacfe311687ba6ae882cab6adc470171a404d4 /extras | |
| parent | 3e2c08d3ea6b90345e626f05dc2f0f77be039188 (diff) | |
Prevent metadata corruption in the race conditions between
FOP->open() and FOP->link().
Backport of http://review.gluster.org/8982
Problem: crypt_open() modifies @local->format, which is used
by crypt_link() to store the updated metadata string on disk.
This results in metadata corruption.
Fixup: Don't modify @local->format by FOP->open(). Instead
modify a local copy, allocated in the low-level meta-data
handler open_format_v1().
Change-Id: Ife5bcb87de6db6d7ba5b65743adce86b2e3da890
BUG: 1169517
Signed-off-by: Edward Shishkin <edward@redhat.com>
Reviewed-on: http://review.gluster.org/9225
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Emmanuel Dreyfus <manu@netbsd.org>
Reviewed-by: Raghavendra Bhat <raghavendra@redhat.com>
Tested-by: Raghavendra Bhat <raghavendra@redhat.com>
Diffstat (limited to 'extras')
0 files changed, 0 insertions, 0 deletions