| From c0dfa2cb7cfcfca131710ef57ae3bff9677ffb31 Mon Sep 17 00:00:00 2001 |
| From: Trond Myklebust <trond.myklebust@hammerspace.com> |
| Date: Thu, 27 Feb 2020 11:01:12 -0500 |
| Subject: [PATCH] NFSv4/pnfs: Return valid stateids in |
| nfs_layout_find_inode_by_stateid() |
| |
| commit d911c57a19551c6bef116a3b55c6b089901aacb0 upstream. |
| |
| Make sure to test the stateid for validity so that we catch instances |
| where the server may have been reusing stateids in |
| nfs_layout_find_inode_by_stateid(). |
| |
| Fixes: 7b410d9ce460 ("pNFS: Delay getting the layout header in CB_LAYOUTRECALL handlers") |
| Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/fs/nfs/callback_proc.c b/fs/nfs/callback_proc.c |
| index 315967354954..bcc51f131a49 100644 |
| --- a/fs/nfs/callback_proc.c |
| +++ b/fs/nfs/callback_proc.c |
| @@ -130,6 +130,8 @@ static struct inode *nfs_layout_find_inode_by_stateid(struct nfs_client *clp, |
| |
| list_for_each_entry_rcu(server, &clp->cl_superblocks, client_link) { |
| list_for_each_entry(lo, &server->layouts, plh_layouts) { |
| + if (!pnfs_layout_is_valid(lo)) |
| + continue; |
| if (stateid != NULL && |
| !nfs4_stateid_match_other(stateid, &lo->plh_stateid)) |
| continue; |
| -- |
| 2.7.4 |
| |