Fix assert vs error codes usage in get_sign_output_length()

Change-Id: Ieca2200d6f844590ed16b0dc314e56a4572c5dfa

diff --git a/src/sign.c b/src/sign.c
index 7dd2a73..3ad2279 100644 (file)
--- a/src/sign.c
+++ b/src/sign.c
@@ -60,9 +60,14 @@ static int get_sign_output_length(const yaca_ctx_h ctx, size_t input_len, size_t
 {
        struct yaca_sign_ctx_s *c = get_sign_ctx(ctx);
 
-       if (c == NULL || c->mdctx == NULL || c->mdctx->pctx == NULL)
+       if (c == NULL)
                return YACA_ERROR_INVALID_ARGUMENT;
 
+       assert(c->mdctx != NULL);
+
+       if (c->mdctx->pctx == NULL)
+               return YACA_ERROR_INTERNAL;
+
        EVP_PKEY *pkey = EVP_PKEY_CTX_get0_pkey(c->mdctx->pctx);
        if (pkey == NULL) {
                ERROR_DUMP(YACA_ERROR_INVALID_ARGUMENT);