[Bugs] [Bug 1311411] New: gfapi : listxattr is broken for handle ops.

[bugzilla at redhat.com]

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

            Bug ID: 1311411
           Summary: gfapi : listxattr is broken for handle ops.
           Product: GlusterFS
           Version: 3.7.9
         Component: libgfapi
          Severity: medium
          Priority: medium
          Assignee: jthottan at redhat.com
          Reporter: jthottan at redhat.com
        QA Contact: sdharane at redhat.com
                CC: bugs at gluster.org, jthottan at redhat.com,
                    kkeithle at redhat.com, ndevos at redhat.com,
                    sdharane at redhat.com, skoduri at redhat.com
        Depends On: 1310620



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

Description of problem:
listxattr is called as part of glfs_h_getxattrs() when NULL is passed to name
field. but glfs_h_getxattrs_common() inside it will error out if name is NULL

--- Additional comment from Vijay Bellur on 2016-02-22 06:34:35 EST ---

REVIEW: http://review.gluster.org/13482 (gfapi : fixing listxattr call for
handle ops) posted (#1) for review on master by jiffin tony Thottan
(jthottan at redhat.com)

--- Additional comment from Vijay Bellur on 2016-02-22 07:24:04 EST ---

REVIEW: http://review.gluster.org/13482 (gfapi : fixing listxattr call for
handle ops) posted (#2) for review on master by jiffin tony Thottan
(jthottan at redhat.com)

--- Additional comment from Vijay Bellur on 2016-02-23 07:49:40 EST ---

COMMIT: http://review.gluster.org/13482 committed in master by Niels de Vos
(ndevos at redhat.com) 
------
commit 0bc54f3ea7a56040df4a67b8626200b47fa3d779
Author: Jiffin Tony Thottan <jthottan at gmail.com>
Date:   Mon Feb 22 16:50:12 2016 +0530

    gfapi : fixing listxattr call for handle ops

    Currently listxattr is called as part of glfs_h_getxattrs() when NULL is
    passed to name field, but glfs_h_getxattrs_common() inside it will error
    out if name is NULL. Therefore listxattr is broken for handle ops.

    Change-Id: I9ced6e33525e1e1a50298972e4922c954fc2b223
    BUG: 1310620
    Signed-off-by: Jiffin Tony Thottan <jthottan at redhat.com>
    Reviewed-on: http://review.gluster.org/13482
    Reviewed-by: Niels de Vos <ndevos 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>


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1310620
[Bug 1310620] gfapi : listxattr is broken for handle ops.
-- 
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=zD8rWmBGPY&a=cc_unsubscribe