commit accfc1cd2e3410e70ad43487b83a718239b1f599
author Koen Zandberg <koen@bergzand.net> Mon Mar 12 17:45:24 2018 +0100
committer Koen Zandberg <koen@bergzand.net> Mon Mar 12 17:54:31 2018 +0100
tree 9f4befd823d735b929b1ec9b8f51e4e27f987b58
parent a2ab6d9bfb1f0242c9fe3d84a10ebcfcc64bb296

Fix error propagation from ECDSA_Sign

src/SignerInfo.c

diff --git a/src/SignerInfo.c b/src/SignerInfo.c
index f1a340e..ddc6811 100644
--- a/src/SignerInfo.c
+++ b/src/SignerInfo.c
@@ -202,19 +202,19 @@
 	switch (alg) {
 #ifdef USE_ECDSA_SHA_256
 	case COSE_Algorithm_ECDSA_SHA_256:
-		f = ECDSA_Sign(&pSigner->m_message, INDEX_SIGNATURE, pSigner->m_pkey, 256, pbToSign, cbToSign, perr);
+		if (!ECDSA_Sign(&pSigner->m_message, INDEX_SIGNATURE, pSigner->m_pkey, 256, pbToSign, cbToSign, perr)) goto errorReturn;
 		break;
 #endif
 
 #ifdef USE_ECDSA_SHA_384
 	case COSE_Algorithm_ECDSA_SHA_384:
-		f = ECDSA_Sign(&pSigner->m_message, INDEX_SIGNATURE, pSigner->m_pkey, 384, pbToSign, cbToSign, perr);
+		if (!ECDSA_Sign(&pSigner->m_message, INDEX_SIGNATURE, pSigner->m_pkey, 384, pbToSign, cbToSign, perr)) goto errorReturn;
 		break;
 #endif
 
 #ifdef USE_ECDSA_SHA_512
 	case COSE_Algorithm_ECDSA_SHA_512:
-		f = ECDSA_Sign(&pSigner->m_message, INDEX_SIGNATURE, pSigner->m_pkey, 512, pbToSign, cbToSign, perr);
+		if (!ECDSA_Sign(&pSigner->m_message, INDEX_SIGNATURE, pSigner->m_pkey, 512, pbToSign, cbToSign, perr)) goto errorReturn;
 		break;
 #endif