[Bugs] [Bug 1332838] rpc: assign port only if it is unreserved

[bugzilla at redhat.com]

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



--- Comment #2 from Vijay Bellur <vbellur at redhat.com> ---
COMMIT: http://review.gluster.org/14204 committed in release-3.7 by Raghavendra
G (rgowdapp at redhat.com) 
------
commit 8f6ab172c9817638f9032ed4436423efd8f8c56c
Author: Prasanna Kumar Kalever <prasanna.kalever at redhat.com>
Date:   Mon Apr 4 16:15:18 2016 +0530

    rpc: assign port only if it is unreserved

    Current order:
        assign port;
        check for port;
        if reserved {
            port--;
            continue to i;
        }
        bind();

    basically, we are assigning port first then checking if it is reserved

    Fix:
        get unreserved port;
        assign port;
        bind();

    from now, we get unreserved port first and then assign it

    Backport of:
    > Change-Id: I004580c5215e7c9cae3594af6405b20fcd9fa4ad
    > BUG: 1323659
    > Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever at redhat.com>
    > Reviewed-on: http://review.gluster.org/13900
    > Tested-by: Prasanna Kumar Kalever <pkalever at redhat.com>
    > NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    > Reviewed-by: Jeff Darcy <jdarcy at redhat.com>
    > CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    > Reviewed-by: Raghavendra Talur <rtalur at redhat.com>
    > Smoke: Gluster Build System <jenkins at build.gluster.com>
    > Reviewed-by: Raghavendra G <rgowdapp at redhat.com>
    > Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever at redhat.com>

    Change-Id: I772c1323606f1853542e22d8ab4d80934871efcb
    BUG: 1332838
    Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever at redhat.com>
    Reviewed-on: http://review.gluster.org/14204
    Tested-by: Prasanna Kumar Kalever <pkalever at redhat.com>
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Raghavendra G <rgowdapp 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=jAfGHGQ5E2&a=cc_unsubscribe