diff options
author | Raghavendra Bhat <raghavendra@redhat.com> | 2018-11-29 12:04:45 -0500 |
---|---|---|
committer | Shyamsundar Ranganathan <srangana@redhat.com> | 2018-12-12 12:59:18 +0000 |
commit | cc0da4931533d21dbc3b9478984e24a7056dc835 (patch) | |
tree | 1c84dc64e7648104f169e96e1ba082c83227c669 /extras/ocf | |
parent | fbdaffdb6d90409124507b3d9b15fc5d6b3ed8e6 (diff) |
features/bitrot: compare the signature with proper length
* The scrubber was comparing the checksum of the file that it
calculated (by reading the file) with the on disk signature
(stored via xattr) wrongly. It was using strlen to calculate
the signature, while the actual length of the signature is
given by the brick. Just use the actual length that the brick
provides instead of trying to calculate the signature length via
strlen API.
* In posix, gfid2path was using the same string that contains the
list of all the xattrs of file to save the value of the gfid2path
xattr as well. This causes confusion when gfid2path xattr is queried
by scrubber for getting the actual path of a corrupted file. Use
separate string to fetch the value of the xattr instead of the string
that contains the list of xattrs.
Backport of:
> Patch: https://review.gluster.org/21752/
> BUG: 1654805
> Change-Id: I2d664ab524d2b312233476cb35863dde3122e9a9
(cherry picked from commit f77fb6d568616592ab25501c402c140d15235ca9)
Change-Id: I2d664ab524d2b312233476cb35863dde3122e9a9
fixes: bz#1654370
Signed-off-by: Raghavendra Bhat <raghavendra@redhat.com>
Diffstat (limited to 'extras/ocf')
0 files changed, 0 insertions, 0 deletions