[Bugs] [Bug 1224222] New: BitRot :- checksum value stored in xattr is different than actual value for some file (checksum is truncated if it has terminating character as part of checksum itself)

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1224222

            Bug ID: 1224222
           Summary: BitRot :- checksum value stored in xattr is different
                    than actual value for some file (checksum is truncated
                    if it has terminating character as part of checksum
                    itself)
           Product: Red Hat Gluster Storage
           Version: 3.1
         Component: glusterfs
     Sub Component: bitrot
          Severity: high
          Assignee: rhs-bugs at redhat.com
          Reporter: rmekala at redhat.com
        QA Contact: rmekala at redhat.com
                CC: bugs at gluster.org, mzywusko at redhat.com,
                    nsathyan at redhat.com, racpatel at redhat.com,
                    vshankar at redhat.com



+++ This bug was initially created as a clone of Bug #1208124 +++

Description of problem:
=======================
If checksum value has terminating character in checksum value, it truncates
remaining part before storing in xattr. Hence checksum value is different then
stored value


Version-Release number of selected component (if applicable):
=============================================================
0.803.gitf64666f.el6.x86_64

How reproducible:
=================
always

Steps to Reproduce:
===================
1. create and mount volume.
2. enable bitrot
3. create few files.
4. verify the checksum for each file

[root at rhs-client44 ~]# sha256sum
/pavanbrick7/r1/rhs-client37.lab.eng.blr.redhat.com/linux-3.19/arch/alpha/include/asm/cache.h
439634a7ce7919c5b05afe05495219a3cbbcf5ced2c45f003c1db8d37a08cbd3 
/pavanbrick7/r1/rhs-client37.lab.eng.blr.redhat.com/linux-3.19/arch/alpha/include/asm/cache.h
[root at rhs-client44 ~]# getfattr -d -m . -e hex
/pavanbrick7/r1/rhs-client37.lab.eng.blr.redhat.com/linux-3.19/arch/alpha/include/asm/cache.h
getfattr: Removing leading '/' from absolute path names
# file:
pavanbrick7/r1/rhs-client37.lab.eng.blr.redhat.com/linux-3.19/arch/alpha/include/asm/cache.h
trusted.gfid=0x3e6e212be7fa4e83861ac4224b008c16
trusted.glusterfs.bit-rot.signature=0x01000000000000000100000000000000439634a7ce7919c5b05afe05495219a3cbbcf5ced2c45f
trusted.glusterfs.bit-rot.version=0x0100000000000000551a6a04000b30ee


checksum has - 00 so remaing part (3c1db8d37a08cbd3) is truncated in
trusted.glusterfs.bit-rot.signature


Actual Result:-
===============
checksum mismatch

--- Additional comment from Venky Shankar on 2015-04-10 06:07:31 EDT ---

Patch[1] contains fixes for this bug which is already merged, hence moving bug
to MODIFIED.

[1]: http://review.gluster.org/10118

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=5DnSsbFeEZ&a=cc_unsubscribe