[132/151] lustre: llite: Add exit for filedata allocation failed

Commit Message

James Simmons Sept. 30, 2019, 6:56 p.m. UTC

From: Ben Evans <[e-mail]>

When the filedata allocation fails, we need to exit to
a later point than out_openerr, which calls
deauthorize_statahead and ll_file_data_put, neither of
which is valid. (This leads to a panic.)

WC-bug-id: https://jira.whamcloud.com/browse/LU-10662
Cray-bug-id: LUS-1321
Lustre-commit: d9edfa50683a ('LU-10662 llite: Add exit for filedata allocation failed")
Signed-off-by: Ben Evans <[e-mail]>
Reviewed-on: https://review.whamcloud.com/31296
Reviewed-by: Patrick Farrell <[e-mail]>
Reviewed-by: Alexandr Boyko <[e-mail]>
Reviewed-by: Oleg Drokin <[e-mail]>
Signed-off-by: James Simmons <[e-mail]>
---
 fs/lustre/llite/file.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Patch

diff --git a/fs/lustre/llite/file.c b/fs/lustre/llite/file.c
index da5bf86..6732b68 100644
--- a/fs/lustre/llite/file.c
+++ b/fs/lustre/llite/file.c
@@ -540,7 +540,7 @@  int ll_file_open(struct inode *inode, struct file *file)
 	fd = ll_file_data_get();
 	if (!fd) {
 		rc = -ENOMEM;
-		goto out_openerr;
+		goto out_nofiledata;
 	}
 
 	fd->fd_file = file;
@@ -723,6 +723,7 @@  int ll_file_open(struct inode *inode, struct file *file)
 		ll_stats_ops_tally(ll_i2sbi(inode), LPROC_LL_OPEN, 1);
 	}
 
+out_nofiledata:
 	if (it && it_disposition(it, DISP_ENQ_OPEN_REF)) {
 		ptlrpc_req_finished(it->it_request);
 		it_clear_disposition(it, DISP_ENQ_OPEN_REF);