From 103f984421afdd7d2cfe134860a1a28cdfaa1c30 Mon Sep 17 00:00:00 2001
From: David Garske <david@wolfssl.com>
Date: Mon, 16 Nov 2015 11:54:23 -0800
Subject: [PATCH] Cleanup of the signature wrapper error cases to be more
 explicit.

---
 wolfcrypt/src/signature.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/wolfcrypt/src/signature.c b/wolfcrypt/src/signature.c
index d069e3491..840dd394a 100644
--- a/wolfcrypt/src/signature.c
+++ b/wolfcrypt/src/signature.c
@@ -110,8 +110,6 @@ int wc_SignatureVerify(
     /* Perform hash of data */
     ret = wc_Hash(hash_type, data, data_len, hash_data, hash_len);
     if(ret == 0) {
-        /* Default to bad argument */
-        ret = BAD_FUNC_ARG;
 
         /* Verify signature using hash as data */
         switch(sig_type) {
@@ -150,6 +148,7 @@ int wc_SignatureVerify(
 
             case WC_SIGNATURE_TYPE_NONE:
             default:
+                ret = BAD_FUNC_ARG;
                 break;
         }
     }
@@ -198,9 +197,6 @@ int wc_SignatureGenerate(
     /* Perform hash of data */
     ret = wc_Hash(hash_type, data, data_len, hash_data, hash_len);
     if (ret == 0) {
-        /* Default to bad argument */
-        ret = BAD_FUNC_ARG;
-
         /* Create signature using hash as data */
         switch(sig_type) {
 #ifdef HAVE_ECC
@@ -223,6 +219,7 @@ int wc_SignatureGenerate(
 
             case WC_SIGNATURE_TYPE_NONE:
             default:
+                ret = BAD_FUNC_ARG;
                 break;
         }
     }