]> www.infradead.org Git - users/hch/misc.git/commitdiff
nfs: fix memory leak in error path of nfs4_do_reclaim
authorLi Lingfeng <lilingfeng3@huawei.com>
Wed, 4 Sep 2024 12:34:57 +0000 (20:34 +0800)
committerAnna Schumaker <anna.schumaker@oracle.com>
Mon, 23 Sep 2024 19:03:12 +0000 (15:03 -0400)
Commit c77e22834ae9 ("NFSv4: Fix a potential sleep while atomic in
nfs4_do_reclaim()") separate out the freeing of the state owners from
nfs4_purge_state_owners() and finish it outside the rcu lock.
However, the error path is omitted. As a result, the state owners in
"freeme" will not be released.
Fix it by adding freeing in the error path.

Fixes: c77e22834ae9 ("NFSv4: Fix a potential sleep while atomic in nfs4_do_reclaim()")
Signed-off-by: Li Lingfeng <lilingfeng3@huawei.com>
Cc: stable@vger.kernel.org # v5.3+
Signed-off-by: Anna Schumaker <anna.schumaker@oracle.com>
fs/nfs/nfs4state.c

index 877f682b45f2d0b2315f635cad5ad8196cb40cc4..30aba1dedaba6cc95a33571b9fcbd3b442648a6f 100644 (file)
@@ -1957,6 +1957,7 @@ restart:
                                set_bit(ops->owner_flag_bit, &sp->so_flags);
                                nfs4_put_state_owner(sp);
                                status = nfs4_recovery_handle_error(clp, status);
+                               nfs4_free_state_owners(&freeme);
                                return (status != 0) ? status : -EAGAIN;
                        }