LU-4423 obdclass: fix return value check in capa_hmac()

In case of error, the function crypto_alloc_hash() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check
should be replaced with IS_ERR().

Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Oleg Drokin <oleg.drokin@intel.com>
Change-Id: I4889387752d1eb5400649cd5f4da172d64c054e2
Reviewed-on: http://review.whamcloud.com/8681
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Emoly Liu <emoly.liu@intel.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>

diff --git a/lustre/obdclass/capa.c b/lustre/obdclass/capa.c
index 5f5bc1b..29c40c1 100644 (file)
--- a/lustre/obdclass/capa.c
+++ b/lustre/obdclass/capa.c
@@ -258,10 +258,10 @@ int capa_hmac(__u8 *hmac, struct lustre_capa *capa, __u8 *key)
         alg = &capa_hmac_algs[capa_alg(capa)];
 
        tfm = crypto_alloc_hash(alg->ha_name, 0, 0);
         alg = &capa_hmac_algs[capa_alg(capa)];
 
        tfm = crypto_alloc_hash(alg->ha_name, 0, 0);

-        if (!tfm) {
+        if (IS_ERR(tfm)) {

                 CERROR("crypto_alloc_tfm failed, check whether your kernel"
                        "has crypto support!\n");
                 CERROR("crypto_alloc_tfm failed, check whether your kernel"
                        "has crypto support!\n");

-                return -ENOMEM;
+                return PTR_ERR(tfm);

         }
         keylen = alg->ha_keylen;
 
         }
         keylen = alg->ha_keylen;