+++ 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/15584 (gfapi: Fix few fd ref leaks) posted (#1) for review on release-3.9 by soumya k (skoduri)
COMMIT: http://review.gluster.org/15584 committed in release-3.9 by Raghavendra Talur (rtalur) ------ commit 6b037e3782ebafe3b08656c578b5fa245f68395f 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: I17dd9e31eca7ffa1f60a9e3a66b1c7051ce9002f BUG: 1379707 Signed-off-by: Soumya Koduri <skoduri> Reviewed-on: http://review.gluster.org/15584 Smoke: Gluster Build System <jenkins.org> NetBSD-regression: NetBSD Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.org> Reviewed-by: Shyamsundar Ranganathan <srangana> Reviewed-by: Raghavendra Talur <rtalur>
This bug is getting closed because GlusterFS-3.9 has reached its end-of-life [1]. Note: This bug is being closed using a script. No verification has been performed to check if it still exists on newer releases of GlusterFS. If this bug still exists in newer GlusterFS releases, please open a new bug against the newer release. [1]: https://www.gluster.org/community/release-schedule/