[Bugs] [Bug 1343286] enabling glusternfs with nfs.rpc-auth-allow to many hosts failed

bugzilla at redhat.com bugzilla at redhat.com
Thu Jun 9 18:40:31 UTC 2016


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



--- Comment #5 from Vijay Bellur <vbellur at redhat.com> ---
COMMIT: http://review.gluster.org/14667 committed in master by Niels de Vos
(ndevos at redhat.com) 
------
commit 3bd4b1529817d4ce40c2b9654044e6230cbf7233
Author: Bipin Kunal <bkunal at redhat.com>
Date:   Tue Jun 7 18:30:16 2016 +0530

    nfs: build exportlist with multiple groupnodes

    The EXPORT procedure of the MOUNT protocol does not correctly create
    structures for the 'groupnodes' in the reply. Each 'groupnode' should be
    a single entry in the 'nfs.rpc-auth-allow' volume option. Because the
    value is handled as a single string, the encoding of the
    groupnode->gr_name fails when the value of the volume option is longer
    than 255 characters.

    In the error case, encoding the EXPORTS reply fails, and the waiting
    'showmount' command will not receive a reply and times out.

    Splitting the allowed entries and creating a groupnode for each one
    prevents the too long ->gr_name. This is following the structures for
    the EXPORTS reply in the MOUNT protocol more correctly as well. Note
    that the contents of ->gr_name is expected to be server dependent.

    Change-Id: Ibbabad581cc9aa00feb80fbbc851a1b10b28383d
    BUG: 1343286
    Signed-off-by: Niels de Vos <ndevos at redhat.com>
    Reviewed-on: http://review.gluster.org/14667
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: bipin kunal <kunalbipin at gmail.com>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    Reviewed-by: jiffin tony Thottan <jthottan at redhat.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.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=1ux7kFJMQ6&a=cc_unsubscribe


More information about the Bugs mailing list