+++ 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/15585 (gfapi: Fix few fd ref leaks) posted (#1) for review on release-3.8 by soumya k (skoduri)
COMMIT: http://review.gluster.org/15585 committed in release-3.8 by Niels de Vos (ndevos) ------ commit 7a50690e8939d1f806556ece1bad9fc2a81c3b80 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) Change-Id: I0919abbddb618d08aaf77b2ce1173eec18b382f1 BUG: 1379708 Signed-off-by: Soumya Koduri <skoduri> Reviewed-on: http://review.gluster.org/15585 Smoke: Gluster Build System <jenkins.org> NetBSD-regression: NetBSD Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.org> Reviewed-by: Niels de Vos <ndevos>
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.8.5, please open a new bug report. glusterfs-3.8.5 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] https://www.gluster.org/pipermail/announce/2016-October/000061.html [2] https://www.gluster.org/pipermail/gluster-users/