This patch moves the '!err' condition into the above else scope,
what is more obvious and has the secondary goal of avoid false-positives
in statical analyze tools.

Signed-off-by: Geyslan G. Bem <geys...@gmail.com>
---
 drivers/crypto/n2_core.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/crypto/n2_core.c b/drivers/crypto/n2_core.c
index e1f0ab4..afc6983 100644
--- a/drivers/crypto/n2_core.c
+++ b/drivers/crypto/n2_core.c
@@ -1511,9 +1511,9 @@ static int __n2_register_one_ahash(const struct 
n2_hash_tmpl *tmpl)
                kfree(p);
        } else {
                pr_info("%s alg registered\n", base->cra_name);
+               if (p->hmac_type != AUTH_TYPE_RESERVED)
+                       err = __n2_register_one_hmac(p);
        }
-       if (!err && p->hmac_type != AUTH_TYPE_RESERVED)
-               err = __n2_register_one_hmac(p);
        return err;
 }
 
-- 
1.8.4

--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to