[Bugs] [Bug 1331264] New: libgfapi:Setting need_lookup on wrong list

[bugzilla at redhat.com]

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

            Bug ID: 1331264
           Summary: libgfapi:Setting need_lookup on wrong list
           Product: GlusterFS
           Version: 3.7.11
         Component: libgfapi
          Keywords: Triaged
          Assignee: bugs at gluster.org
          Reporter: rkavunga at redhat.com
        QA Contact: sdharane at redhat.com
                CC: bugs at gluster.org, sdharane at redhat.com
        Depends On: 1330476



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

Description of problem:

We are setting need_lookup on wrong list during readdirp cbk. we are using the
previous list .
Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

--- Additional comment from Vijay Bellur on 2016-04-26 06:19:44 EDT ---

REVIEW: http://review.gluster.org/14073 (gfapi: set need_lookup flag on
response list) posted (#1) for review on master by mohammed rafi  kc
(rkavunga at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-26 11:36:34 EDT ---

REVIEW: http://review.gluster.org/14073 (gfapi: set need_lookup flag on
response list) posted (#2) for review on master by Niels de Vos
(ndevos at redhat.com)

--- Additional comment from Vijay Bellur on 2016-04-27 10:26:47 EDT ---

COMMIT: http://review.gluster.org/14073 committed in master by Niels de Vos
(ndevos at redhat.com) 
------
commit 537557da59876536d33cd25a8ef485e7b5fe8849
Author: Mohammed Rafi KC <rkavunga at redhat.com>
Date:   Tue Apr 26 14:52:29 2016 +0530

    gfapi: set need_lookup flag on response list

    We set need_lookup flag for entries returned by readdirp
    to force lookup. Currently we are setting on the previously
    stored list, rather than response list returned by readdirp.

    This patch will iterate over current list returned by readdirp
    and will set need_lookup flag.

    Change-Id: Ibd6fcbc188f4c87f40ece7a9dcda27645401c240
    BUG: 1330476
    Signed-off-by: Mohammed Rafi KC <rkavunga at redhat.com>
    Reviewed-on: http://review.gluster.org/14073
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    Reviewed-by: Niels de Vos <ndevos at redhat.com>
    Reviewed-by: Rajesh Joseph <rjoseph at redhat.com>
    Smoke: Gluster Build System <jenkins at build.gluster.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.com>


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1330476
[Bug 1330476] libgfapi:Setting need_lookup on wrong list
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.