diff options
| author | David Howells <dhowells@redhat.com> | 2023-09-29 15:35:17 +0200 |
|---|---|---|
| committer | David Howells <dhowells@redhat.com> | 2023-12-28 10:45:20 +0100 |
| commit | 6ba22d8d1521f35ca1343e64f69d7857f0340e5e (patch) | |
| tree | c10a5b0a157ee6f183e71d0810f7c9ac21ddf704 /fs | |
| parent | netfs: Add a hook to allow tell the netfs to update its i_size (diff) | |
| download | linux-6ba22d8d1521f35ca1343e64f69d7857f0340e5e.tar.xz linux-6ba22d8d1521f35ca1343e64f69d7857f0340e5e.zip | |
netfs: Make netfs_put_request() handle a NULL pointer
Make netfs_put_request() just return if given a NULL request pointer.
Signed-off-by: David Howells <dhowells@redhat.com>
Reviewed-by: Jeff Layton <jlayton@kernel.org>
cc: linux-cachefs@redhat.com
cc: linux-fsdevel@vger.kernel.org
cc: linux-mm@kvack.org
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/netfs/objects.c | 23 |
1 files changed, 13 insertions, 10 deletions
diff --git a/fs/netfs/objects.c b/fs/netfs/objects.c index 65a17dd4ab49..3aa0bfbc04ec 100644 --- a/fs/netfs/objects.c +++ b/fs/netfs/objects.c @@ -106,19 +106,22 @@ static void netfs_free_request(struct work_struct *work) void netfs_put_request(struct netfs_io_request *rreq, bool was_async, enum netfs_rreq_ref_trace what) { - unsigned int debug_id = rreq->debug_id; + unsigned int debug_id; bool dead; int r; - dead = __refcount_dec_and_test(&rreq->ref, &r); - trace_netfs_rreq_ref(debug_id, r - 1, what); - if (dead) { - if (was_async) { - rreq->work.func = netfs_free_request; - if (!queue_work(system_unbound_wq, &rreq->work)) - BUG(); - } else { - netfs_free_request(&rreq->work); + if (rreq) { + debug_id = rreq->debug_id; + dead = __refcount_dec_and_test(&rreq->ref, &r); + trace_netfs_rreq_ref(debug_id, r - 1, what); + if (dead) { + if (was_async) { + rreq->work.func = netfs_free_request; + if (!queue_work(system_unbound_wq, &rreq->work)) + BUG(); + } else { + netfs_free_request(&rreq->work); + } } } } |