gfs2: Fix refcount leak in gfs2_glock_poke

In gfs2_glock_poke, make sure gfs2_holder_uninit is called on the local
glock holder.  Without that, we're leaking a glock and a pid reference.

Fixes: 9e8990dea926 ("gfs2: Smarter iopen glock waiting")
Cc: stable@vger.kernel.org # v5.8+
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>

diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c
index 57134d326cfa17f963e6526a6fdbf3ac6eb8c304..f13b136654cae0acc6812af38bcd7771d896f967 100644 (file)
--- a/fs/gfs2/glock.c
+++ b/fs/gfs2/glock.c
@@ -790,9 +790,11 @@ static void gfs2_glock_poke(struct gfs2_glock *gl)
        struct gfs2_holder gh;
        int error;
 
-       error = gfs2_glock_nq_init(gl, LM_ST_SHARED, flags, &gh);
+       gfs2_holder_init(gl, LM_ST_SHARED, flags, &gh);
+       error = gfs2_glock_nq(&gh);
        if (!error)
                gfs2_glock_dq(&gh);
+       gfs2_holder_uninit(&gh);
 }
 
 static bool gfs2_try_evict(struct gfs2_glock *gl)