Unify various EVP_*{Update,Final}*() wrappers

The correct way of wrapping foo() is 'int ret; ret = foo(); return ret;'
because 'return foo();' would be too simple... Also unify branching from
EVP_Cipher* into EVP_Encrypt* EVP_Decrypt*.

1 files changed, 10 insertions, 18 deletions

diff --git a/src/lib/libcrypto/evp/evp_enc.c b/src/lib/libcrypto/evp/evp_enc.c
index 0867070a76..04aa8f57a7 100644
--- a/src/lib/libcrypto/evp/evp_enc.c
+++ b/src/lib/libcrypto/evp/evp_enc.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: evp_enc.c,v 1.56 2023/11/29 21:35:57 tb Exp $ */
+/* $OpenBSD: evp_enc.c,v 1.57 2023/12/01 06:53:18 tb Exp $ */
 /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
  * All rights reserved.
  *
@@ -198,8 +198,8 @@ EVP_CipherUpdate(EVP_CIPHER_CTX *ctx, unsigned char *out, int *outl,
 {
         if (ctx->encrypt)
                 return EVP_EncryptUpdate(ctx, out, outl, in, inl);
-        else
-                return EVP_DecryptUpdate(ctx, out, outl, in, inl);
+
+        return EVP_DecryptUpdate(ctx, out, outl, in, inl);
 }
 
 int
@@ -207,8 +207,8 @@ EVP_CipherFinal_ex(EVP_CIPHER_CTX *ctx, unsigned char *out, int *outl)
 {
         if (ctx->encrypt)
                 return EVP_EncryptFinal_ex(ctx, out, outl);
-        else
-                return EVP_DecryptFinal_ex(ctx, out, outl);
+
+        return EVP_DecryptFinal_ex(ctx, out, outl);
 }
 
 __warn_references(EVP_CipherFinal,
@@ -217,12 +217,10 @@ __warn_references(EVP_CipherFinal,
 int
 EVP_CipherFinal(EVP_CIPHER_CTX *ctx, unsigned char *out, int *outl)
 {
-        int ret;
         if (ctx->encrypt)
-                ret = EVP_EncryptFinal_ex(ctx, out, outl);
-        else
-                ret = EVP_DecryptFinal_ex(ctx, out, outl);
-        return ret;
+                return EVP_EncryptFinal_ex(ctx, out, outl);
+
+        return EVP_DecryptFinal_ex(ctx, out, outl);
 }
 
 int
@@ -341,10 +339,7 @@ __warn_references(EVP_EncryptFinal,
 int
 EVP_EncryptFinal(EVP_CIPHER_CTX *ctx, unsigned char *out, int *outl)
 {
-        int ret;
-
-        ret = EVP_EncryptFinal_ex(ctx, out, outl);
-        return ret;
+        return EVP_EncryptFinal_ex(ctx, out, outl);
 }
 
 int
@@ -469,10 +464,7 @@ __warn_references(EVP_DecryptFinal,
 int
 EVP_DecryptFinal(EVP_CIPHER_CTX *ctx, unsigned char *out, int *outl)
 {
-        int ret;
-
-        ret = EVP_DecryptFinal_ex(ctx, out, outl);
-        return ret;
+        return EVP_DecryptFinal_ex(ctx, out, outl);
 }
 
 int