Description of problem: A superficial glance while reviewing http://review.gluster.org/#/c/7978/ revealed two memory leaks. A more comprehensive code review is needed. Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info:
I found more leaks/corruption upon further review. Will be using the same bug to send the fix. Pranith
REVIEW: http://review.gluster.org/7982 (features/gfid-access: Fix memory leaks.) posted (#1) for review on master by Raghavendra G (rgowdapp)
REVIEW: http://review.gluster.org/7982 (features/gfid-access: Fix memory leaks.) posted (#2) for review on master by Raghavendra G (rgowdapp)
REVIEW: http://review.gluster.org/7983 (features/gfid-access: Fix memory leaks.) posted (#1) for review on release-3.5 by Raghavendra G (rgowdapp)
REVIEW: http://review.gluster.org/8009 (features/gfid-access: Fix inode leaks and loc path corruption) posted (#1) for review on master by Pranith Kumar Karampuri (pkarampu)
REVIEW: http://review.gluster.org/7982 (features/gfid-access: Fix memory leaks.) posted (#3) for review on master by Raghavendra G (rgowdapp)
REVIEW: http://review.gluster.org/7983 (features/gfid-access: Fix memory leaks.) posted (#2) for review on release-3.5 by Raghavendra G (rgowdapp)
REVIEW: http://review.gluster.org/7983 (features/gfid-access: Fix memory leaks.) posted (#3) for review on release-3.5 by Raghavendra G (rgowdapp)
REVIEW: http://review.gluster.org/7982 (features/gfid-access: Fix memory leaks.) posted (#4) for review on master by Raghavendra G (rgowdapp)
COMMIT: http://review.gluster.org/7982 committed in master by Vijay Bellur (vbellur) ------ commit f6262b543f0eaa08c2d69310bf2353c9cba009e6 Author: Raghavendra G <rgowdapp> Date: Thu Jun 5 10:06:47 2014 +0530 features/gfid-access: Fix memory leaks. Change-Id: I90f6cdb1c8c4face1bb72a9cc77818d308389e45 BUG: 1104919 Signed-off-by: Raghavendra G <rgowdapp> Reviewed-on: http://review.gluster.org/7982 Reviewed-by: Pranith Kumar Karampuri <pkarampu> Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Venky Shankar <vshankar> Reviewed-by: Vijay Bellur <vbellur>
COMMIT: http://review.gluster.org/7983 committed in release-3.5 by Niels de Vos (ndevos) ------ commit b167a0ca7f418abd0ff08cf0332593b00e27caee Author: Raghavendra G <rgowdapp> Date: Thu Jun 5 10:06:47 2014 +0530 features/gfid-access: Fix memory leaks. Change-Id: I90f6cdb1c8c4face1bb72a9cc77818d308389e45 BUG: 1104919 Signed-off-by: Raghavendra G <rgowdapp> Reviewed-on: http://review.gluster.org/7983 Reviewed-by: Pranith Kumar Karampuri <pkarampu> Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Niels de Vos <ndevos>
COMMIT: http://review.gluster.org/8009 committed in master by Vijay Bellur (vbellur) ------ commit 99781d55f16373e793138c7b011eea09a62075b9 Author: Pranith Kumar K <pkarampu> Date: Sun Jun 8 11:23:26 2014 +0530 features/gfid-access: Fix inode leaks and loc path corruption Change-Id: Ib506db28415d57b344aa90e07e74666ad2063c64 BUG: 1104919 Signed-off-by: Pranith Kumar K <pkarampu> Reviewed-on: http://review.gluster.org/8009 Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Kotresh HR <khiremat> Reviewed-by: Vijay Bellur <vbellur>
REVIEW: http://review.gluster.org/8028 (features/gfid-access: Fix inode leaks and loc path corruption) posted (#1) for review on release-3.5 by Pranith Kumar Karampuri (pkarampu)
The second (and last?) Beta for GlusterFS 3.5.1 has been released [1]. Please verify if the release solves this bug report for you. In case the glusterfs-3.5.1beta2 release does not have a resolution for this issue, leave a comment in this bug and move the status to ASSIGNED. If this release fixes the problem for you, leave a note and change the status to VERIFIED. Packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update (possibly an "updates-testing" repository) infrastructure for your distribution. [1] http://supercolony.gluster.org/pipermail/gluster-users/2014-June/040547.html [2] http://supercolony.gluster.org/pipermail/gluster-users/
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.5.1, please reopen this bug report. glusterfs-3.5.1 has been announced on the Gluster Users mailinglist [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution. [1] http://supercolony.gluster.org/pipermail/gluster-users/2014-June/040723.html [2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user