[Bugs] [Bug 1140549] DHT: Rebalance process crash after add-brick and `rebalance start' operation

[bugzilla at redhat.com]

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



--- Comment #4 from Anand Avati <aavati at redhat.com> ---
COMMIT: http://review.gluster.org/8732 committed in release-3.5 by Niels de Vos
(ndevos at redhat.com) 
------
commit 8c6eb5e04f837f679b3ca04ef784054afb18d66c
Author: Shyam <srangana at redhat.com>
Date:   Thu Sep 11 14:21:23 2014 +0530

    cluster/dht: Fixed double UNWIND in lookup everywhere code

    In dht_lookup_everywhere_done:
    Line: 1194 we call DHT_STACK_UNWIND
    and in the same if condition we go ahead and call, goto
    unwind_hashed_and_cached; which at Line 1371 calls another UNWIND.

    As is obvious, higher frames could cleanup their locals and on
    receiving the next unwind could cause a coredump of the process.

    Fixed the same by calling the required return post the first unwind

    Change-Id: Ic5d57da98255b8616a65b4caaedabeba9144fd49
    BUG: 1140549
    Signed-off-by: Shyam <srangana at redhat.com>
    Reviewed-on: http://review.gluster.org/8666
    Reviewed-by: N Balachandran <nbalacha at redhat.com>
    Reviewed-by: Vijay Bellur <vbellur at redhat.com>
    Reviewed-by: Jeff Darcy <jdarcy at redhat.com>
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: susant palai <spalai at redhat.com>
    (cherry picked from commit b3314ea6e820fb659255d0e6e9a32ea259b7526d)
    Signed-off-by: Nithya Balachandran <nbalacha at redhat.com>
    Reviewed-on: http://review.gluster.org/8732
    Reviewed-by: Niels de Vos <ndevos 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=dj27gVUdUF&a=cc_unsubscribe