From ea4be7ba6fcbd8ba6860f3fc7234a51c474f4e30 Mon Sep 17 00:00:00 2001
From: Christian Grothoff <christian@grothoff.org>
Date: Sat, 12 Feb 2022 11:42:25 +0100
Subject: -swap argument/rval for nicer code

---
 src/util/crypto_helper_rsa.c | 50 ++++++++++++++++++++------------------------
 1 file changed, 23 insertions(+), 27 deletions(-)

(limited to 'src/util/crypto_helper_rsa.c')

diff --git a/src/util/crypto_helper_rsa.c b/src/util/crypto_helper_rsa.c
index 75fe33c03..d3f498c07 100644
--- a/src/util/crypto_helper_rsa.c
+++ b/src/util/crypto_helper_rsa.c
@@ -387,18 +387,17 @@ more:
 }
 
 
-struct TALER_BlindedDenominationSignature
+enum TALER_ErrorCode
 TALER_CRYPTO_helper_rsa_sign (
   struct TALER_CRYPTO_RsaDenominationHelper *dh,
   const struct TALER_RsaPubHashP *h_rsa,
   const void *msg,
   size_t msg_size,
-  enum TALER_ErrorCode *ec)
+  struct TALER_BlindedDenominationSignature *bs)
 {
-  struct TALER_BlindedDenominationSignature ds = {
-    .cipher = TALER_DENOMINATION_INVALID
-  };
+  enum TALER_ErrorCode ec = TALER_EC_INVALID;
 
+  bs->cipher = TALER_DENOMINATION_INVALID;
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
               "Starting signature process\n");
   if (GNUNET_OK !=
@@ -406,8 +405,7 @@ TALER_CRYPTO_helper_rsa_sign (
   {
     GNUNET_log (GNUNET_ERROR_TYPE_WARNING,
                 "Failed to connect to helper\n");
-    *ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_UNAVAILABLE;
-    return ds;
+    return TALER_EC_EXCHANGE_DENOMINATION_HELPER_UNAVAILABLE;
   }
 
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
@@ -432,8 +430,7 @@ TALER_CRYPTO_helper_rsa_sign (
       GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING,
                            "send");
       do_disconnect (dh);
-      *ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_UNAVAILABLE;
-      return ds;
+      return TALER_EC_EXCHANGE_DENOMINATION_HELPER_UNAVAILABLE;
     }
   }
 
@@ -446,7 +443,6 @@ TALER_CRYPTO_helper_rsa_sign (
       = (const struct GNUNET_MessageHeader *) buf;
     bool finished = false;
 
-    *ec = TALER_EC_INVALID;
     while (1)
     {
       uint16_t msize;
@@ -466,20 +462,20 @@ TALER_CRYPTO_helper_rsa_sign (
         {
           GNUNET_assert (finished);
           GNUNET_assert (0 == off);
-          return ds;
+          return ec;
         }
         GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING,
                              "recv");
         do_disconnect (dh);
-        *ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_UNAVAILABLE;
+        ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_UNAVAILABLE;
         break;
       }
       if (0 == ret)
       {
         GNUNET_break (0 == off);
         if (! finished)
-          *ec = TALER_EC_EXCHANGE_SIGNKEY_HELPER_BUG;
-        return ds;
+          ec = TALER_EC_EXCHANGE_SIGNKEY_HELPER_BUG;
+        return ec;
       }
       off += ret;
 more:
@@ -495,14 +491,14 @@ more:
         {
           GNUNET_break_op (0);
           do_disconnect (dh);
-          *ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
+          ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
           goto end;
         }
         if (finished)
         {
           GNUNET_break_op (0);
           do_disconnect (dh);
-          *ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
+          ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
           goto end;
         }
         {
@@ -517,15 +513,15 @@ more:
           {
             GNUNET_break_op (0);
             do_disconnect (dh);
-            *ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
+            ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
             goto end;
           }
           GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
                       "Received signature\n");
-          *ec = TALER_EC_NONE;
+          ec = TALER_EC_NONE;
           finished = true;
-          ds.cipher = TALER_DENOMINATION_RSA;
-          ds.details.blinded_rsa_signature = rsa_signature;
+          bs->cipher = TALER_DENOMINATION_RSA;
+          bs->details.blinded_rsa_signature = rsa_signature;
           break;
         }
       case TALER_HELPER_RSA_MT_RES_SIGN_FAILURE:
@@ -533,14 +529,14 @@ more:
         {
           GNUNET_break_op (0);
           do_disconnect (dh);
-          *ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
+          ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
           goto end;
         }
         {
           const struct TALER_CRYPTO_SignFailure *sf =
             (const struct TALER_CRYPTO_SignFailure *) buf;
 
-          *ec = (enum TALER_ErrorCode) ntohl (sf->ec);
+          ec = (enum TALER_ErrorCode) ntohl (sf->ec);
           GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
                       "Signing failed!\n");
           finished = true;
@@ -555,7 +551,7 @@ more:
         {
           GNUNET_break_op (0);
           do_disconnect (dh);
-          *ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
+          ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
           goto end;
         }
         break; /* while(1) loop ensures we recvfrom() again */
@@ -568,7 +564,7 @@ more:
         {
           GNUNET_break_op (0);
           do_disconnect (dh);
-          *ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
+          ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
           goto end;
         }
         break; /* while(1) loop ensures we recvfrom() again */
@@ -583,7 +579,7 @@ more:
                     "Received unexpected message of type %u\n",
                     ntohs (hdr->type));
         do_disconnect (dh);
-        *ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
+        ec = TALER_EC_EXCHANGE_DENOMINATION_HELPER_BUG;
         goto end;
       }
       memmove (buf,
@@ -594,8 +590,8 @@ more:
     } /* while(1) */
 end:
     if (finished)
-      TALER_blinded_denom_sig_free (&ds);
-    return ds;
+      TALER_blinded_denom_sig_free (bs);
+    return ec;
   }
 }
 
-- 
cgit v1.2.3