LU-15598 tgt: free allocated page on error

Free allocated page if cfs_crypto_hash_init() fails.

Fixes: b1e7be00cb6e ("LU-10472 osc: add T10PI support for RPC checksum")
Signed-off-by: Andreas Dilger <adilger@whamcloud.com>
Change-Id: I0a45a82a57a98ad2517ccf50a2be1e8d65550bb5
Reviewed-on: https://review.whamcloud.com/46659
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Reviewed-by: Li Dongyang <dongyangli@ddn.com>

diff --git a/lustre/target/tgt_handler.c b/lustre/target/tgt_handler.c
index c42c74b..c763e0a 100644 (file)
--- a/lustre/target/tgt_handler.c
+++ b/lustre/target/tgt_handler.c
@@ -2046,9 +2046,10 @@ static int tgt_checksum_niobuf_t10pi(struct lu_target *tgt,
 
        req = cfs_crypto_hash_init(cfs_alg, NULL, 0);
        if (IS_ERR(req)) {
-               CERROR("%s: unable to initialize checksum hash %s\n",
-                      tgt_name(tgt), cfs_crypto_hash_name(cfs_alg));
-               return PTR_ERR(req);
+               rc = PTR_ERR(req);
+               CERROR("%s: unable to initialize checksum hash %s: rc = %d\n",
+                      tgt_name(tgt), cfs_crypto_hash_name(cfs_alg), rc);
+               goto out;
        }
 
        buffer = kmap(__page);