[Bugs] [Bug 1360556] afr coverity fixes

bugzilla at redhat.com bugzilla at redhat.com
Thu Jul 28 13:54:52 UTC 2016


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



--- Comment #2 from Vijay Bellur <vbellur at redhat.com> ---
COMMIT: http://review.gluster.org/15018 committed in release-3.8 by Pranith
Kumar Karampuri (pkarampu at redhat.com) 
------
commit 823eb274a3c4226aea44f6feb955a5df04aae190
Author: Ravishankar N <ravishankar at redhat.com>
Date:   Tue Jul 12 10:07:48 2016 +0530

    afr: some coverity fixes

    Note: This is a backport of http://review.gluster.org/14895.
    It contains:
    i) fixes that prevent deadlocks (afr-common.c).
    ii) fixes over-writing op-errno=ENOMEM with possible other values
    (afr-inode-read.c).
    iii) prevents doing further operations with a NULL dictionary if
    allocation fails (afr-self-heal-data.c).
    iv) prevents falsely marking a sink as healed if metadata heal fails
    midway(afr-self-heal-metadata.c).
    v) other minor fixes.

    Considering the above are not trivial fixes, the patch is a good
    candidate for merging in 3.8 branch.

    Thanks to Krutika for a cleaner way to track inode refs in
    afr_set_split_brain_choice().

    Change-Id: I2d968d05b815ad764b7e3f8aa9ad95a792b3c1df
    BUG: 1360556
    Signed-off-by: Ravishankar N <ravishankar at redhat.com>
    Reviewed-on: http://review.gluster.org/15018
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    Reviewed-by: Krutika Dhananjay <kdhananj at redhat.com>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu at redhat.com>

-- 
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=DxgDqXRZvG&a=cc_unsubscribe


More information about the Bugs mailing list