Hide Forgot
+++ This bug was initially created as a clone of Bug #1379285 +++ Description of problem: From the code inspection, have observed that there are fd ref leaks for few fd-based gfapi fops. 'glfs_resolve_fd' returns a fd (either existing or migrated) with an extra ref taken. This needs to be unref'ed at the end of the operation. 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 Worker Ant on 2016-09-26 11:35:39 CEST --- REVIEW: http://review.gluster.org/15573 (gfapi: Fix few fd ref leaks) posted (#1) for review on master by soumya k (skoduri) --- Additional comment from Worker Ant on 2016-09-27 08:43:38 CEST --- COMMIT: http://review.gluster.org/15573 committed in master by Niels de Vos (ndevos) ------ commit d27cffab670858d7812bbb458a0833303d009b3b Author: Soumya Koduri <skoduri> Date: Mon Sep 26 14:54:27 2016 +0530 gfapi: Fix few fd ref leaks From the code inspection, have observed that there are fd ref leaks for few fd-based gfapi fops. 'glfs_resolve_fd' returns a fd (either existing or migrated) with an extra ref taken. This needs to be unref'ed at the end of the operation. Change-Id: Id63394e3e7deafb0c8b06444f2ae847248b126db BUG: 1379285 Signed-off-by: Soumya Koduri <skoduri> Reviewed-on: http://review.gluster.org/15573 NetBSD-regression: NetBSD Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.org> Reviewed-by: Niels de Vos <ndevos> Smoke: Gluster Build System <jenkins.org>
REVIEW: http://review.gluster.org/15604 (gfapi: Fix few fd ref leaks) posted (#1) for review on release-3.7 by soumya k (skoduri)
COMMIT: http://review.gluster.org/15604 committed in release-3.7 by Niels de Vos (ndevos) ------ commit 401aec64b1aec4215651e9d04f264a83433dd06c Author: Soumya Koduri <skoduri> Date: Mon Sep 26 14:54:27 2016 +0530 gfapi: Fix few fd ref leaks From the code inspection, have observed that there are fd ref leaks for few fd-based gfapi fops. 'glfs_resolve_fd' returns a fd (either existing or migrated) with an extra ref taken. This needs to be unref'ed at the end of the operation. Mainline reference: > Change-Id: Id63394e3e7deafb0c8b06444f2ae847248b126db > BUG: 1379285 > Signed-off-by: Soumya Koduri <skoduri> > Reviewed-on: http://review.gluster.org/15573 > Reviewed-by: Niels de Vos <ndevos> (cherry picked from commit d27cffab670858d7812bbb458a0833303d009b3b) BUG: 1379710 Change-Id: I143532a8a14febc8c4aad7a18b1d9166529b30cc Signed-off-by: Soumya Koduri <skoduri> Reviewed-on: http://review.gluster.org/15604 Reviewed-by: Niels de Vos <ndevos> NetBSD-regression: NetBSD Build System <jenkins.org> Smoke: Gluster Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.org>
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.7.17, please open a new bug report. glusterfs-3.7.17 has been announced on the Gluster mailinglists [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://www.gluster.org/pipermail/gluster-devel/2016-November/051414.html [2] https://www.gluster.org/pipermail/gluster-users/