Fix yaca_sign_initialize_hmac/cmac memory leak.

Fix sign example indentation.

Change-Id: I65664d815ebff5eb5861d24a649ea94c9910915f

diff --git a/examples/sign.c b/examples/sign.c
index df7ff58943215325ac0b2b33261185155079d0fd..db6f39a15067aba0f20480944a5779bc47d29996 100644 (file)
--- a/examples/sign.c
+++ b/examples/sign.c
@@ -50,22 +50,22 @@ void simple_sign_verify_asym(yaca_key_type_e type, const char *algo)
 
        /* SIGN */
        if (yaca_simple_calculate_signature(YACA_DIGEST_SHA512,
-                     prv,
-                     lorem4096,
-                     LOREM4096_SIZE,
-                     &signature,
-                     &signature_len) != YACA_ERROR_NONE)
+                                           prv,
+                                           lorem4096,
+                                           LOREM4096_SIZE,
+                                           &signature,
+                                           &signature_len) != YACA_ERROR_NONE)
                goto exit;
 
        dump_hex(signature, signature_len, "[Simple API] %s Signature of lorem4096:", algo);
 
        /* VERIFY */
        if (yaca_simple_verify_signature(YACA_DIGEST_SHA512,
-                       pub,
-                       lorem4096,
-                       LOREM4096_SIZE,
-                       signature,
-                       signature_len) != YACA_ERROR_NONE)
+                                        pub,
+                                        lorem4096,
+                                        LOREM4096_SIZE,
+                                        signature,
+                                        signature_len) != YACA_ERROR_NONE)
                printf("[Simple API] %s verification failed\n", algo);
        else
                printf("[Simple API] %s verification successful\n", algo);
@@ -90,22 +90,22 @@ void simple_sign_verify_hmac(void)
 
        /* SIGN */
        if (yaca_simple_calculate_hmac(YACA_DIGEST_SHA512,
-                     key,
-                     lorem4096,
-                     LOREM4096_SIZE,
-                     &signature1,
-                     &signature_len) != YACA_ERROR_NONE)
+                                      key,
+                                      lorem4096,
+                                      LOREM4096_SIZE,
+                                      &signature1,
+                                      &signature_len) != YACA_ERROR_NONE)
                goto exit;
 
        dump_hex(signature1, signature_len, "[Simple API] HMAC Signature of lorem4096:");
 
        /* VERIFY */
        if (yaca_simple_calculate_hmac(YACA_DIGEST_SHA512,
-                     key,
-                     lorem4096,
-                     LOREM4096_SIZE,
-                     &signature2,
-                     &signature_len) != YACA_ERROR_NONE)
+                                      key,
+                                      lorem4096,
+                                      LOREM4096_SIZE,
+                                      &signature2,
+                                      &signature_len) != YACA_ERROR_NONE)
                goto exit;
 
        if (yaca_memcmp(signature1, signature2, signature_len) != YACA_ERROR_NONE)
@@ -133,11 +133,11 @@ void simple_sign_verify_cmac(void)
 
        /* SIGN */
        if (yaca_simple_calculate_cmac(YACA_ENCRYPT_AES,
-                     key,
-                     lorem4096,
-                     LOREM4096_SIZE,
-                     &signature1,
-                     &signature_len) != YACA_ERROR_NONE)
+                                      key,
+                                      lorem4096,
+                                      LOREM4096_SIZE,
+                                      &signature1,
+                                      &signature_len) != YACA_ERROR_NONE)
                goto exit;
 
        dump_hex(signature1, signature_len, "[Simple API] CMAC Signature of lorem4096:");
@@ -145,11 +145,11 @@ void simple_sign_verify_cmac(void)
 
        /* VERIFY */
        if (yaca_simple_calculate_cmac(YACA_ENCRYPT_AES,
-                     key,
-                     lorem4096,
-                     LOREM4096_SIZE,
-                     &signature2,
-                     &signature_len) != YACA_ERROR_NONE)
+                                      key,
+                                      lorem4096,
+                                      LOREM4096_SIZE,
+                                      &signature2,
+                                      &signature_len) != YACA_ERROR_NONE)
                goto exit;
 
        if (yaca_memcmp(signature1, signature2, signature_len) != YACA_ERROR_NONE)

diff --git a/src/sign.c b/src/sign.c
index aaa5863290f40967ebff894499b0c64a41ec782e..79d77596c8637fd57166bc1d562a1d04b16bbfea 100644 (file)
--- a/src/sign.c
+++ b/src/sign.c
@@ -343,8 +343,6 @@ API int yaca_sign_initialize_hmac(yaca_context_h *ctx,
                goto exit;
        }
 
-       pkey = NULL;
-
        *ctx = (yaca_context_h)nc;
        nc = NULL;
        ret = YACA_ERROR_NONE;
@@ -427,8 +425,6 @@ API int yaca_sign_initialize_cmac(yaca_context_h *ctx,
                goto exit;
        }
 
-       pkey = NULL;
-
        *ctx = (yaca_context_h)nc;
        nc = NULL;
        ret = YACA_ERROR_NONE;