Add the peer cert buffer and count to the X509_STORE_CTX used for the verify callback. Fixes #627.

This commit is contained in:
David Garske
2016-11-22 11:45:00 -08:00
parent b61e6e1219
commit 5b76a37234
3 changed files with 17 additions and 7 deletions

View File

@@ -6861,6 +6861,7 @@ static int DoCertificate(WOLFSSL* ssl, byte* input, word32* inOutIdx,
return MEMORY_E; return MEMORY_E;
} }
#endif #endif
XMEMSET(store, 0, sizeof(WOLFSSL_X509_STORE_CTX));
if (anyError != 0 && ret == 0) if (anyError != 0 && ret == 0)
ret = anyError; ret = anyError;
@@ -6879,6 +6880,8 @@ static int DoCertificate(WOLFSSL* ssl, byte* input, word32* inOutIdx,
store->discardSessionCerts = 0; store->discardSessionCerts = 0;
store->domain = domain; store->domain = domain;
store->userCtx = ssl->verifyCbCtx; store->userCtx = ssl->verifyCbCtx;
store->certs = certs;
store->totalCerts = totalCerts;
#ifdef KEEP_PEER_CERT #ifdef KEEP_PEER_CERT
store->current_cert = &ssl->peerCert; store->current_cert = &ssl->peerCert;
#else #else
@@ -6916,6 +6919,8 @@ static int DoCertificate(WOLFSSL* ssl, byte* input, word32* inOutIdx,
store->discardSessionCerts = 0; store->discardSessionCerts = 0;
store->domain = domain; store->domain = domain;
store->userCtx = ssl->verifyCbCtx; store->userCtx = ssl->verifyCbCtx;
store->certs = certs;
store->totalCerts = totalCerts;
#ifdef KEEP_PEER_CERT #ifdef KEEP_PEER_CERT
store->current_cert = &ssl->peerCert; store->current_cert = &ssl->peerCert;
#endif #endif

View File

@@ -175,6 +175,8 @@ typedef struct WOLFSSL_X509_STORE_CTX {
int error; /* current error */ int error; /* current error */
int error_depth; /* cert depth for this error */ int error_depth; /* cert depth for this error */
int discardSessionCerts; /* so verify callback can flag for discard */ int discardSessionCerts; /* so verify callback can flag for discard */
int totalCerts; /* number of peer cert buffers */
struct buffer* certs; /* peer certs */
} WOLFSSL_X509_STORE_CTX; } WOLFSSL_X509_STORE_CTX;

View File

@@ -371,7 +371,7 @@ static INLINE WC_NORETURN void err_sys(const char* msg)
* msg pointer can be null even when hardcoded and then it won't exit, * msg pointer can be null even when hardcoded and then it won't exit,
* making null pointer checks above the err_sys() call useless. * making null pointer checks above the err_sys() call useless.
* We could just always exit() but some compilers will complain about no * We could just always exit() but some compilers will complain about no
* possible return, with gcc we know the attribute to handle that with * possible return, with gcc we know the attribute to handle that with
* WC_NORETURN. */ * WC_NORETURN. */
if (msg) if (msg)
#endif #endif
@@ -1143,17 +1143,20 @@ static INLINE int myVerify(int preverify, WOLFSSL_X509_STORE_CTX* store)
wolfSSL_X509_get_issuer_name(peer), 0, 0); wolfSSL_X509_get_issuer_name(peer), 0, 0);
char* subject = wolfSSL_X509_NAME_oneline( char* subject = wolfSSL_X509_NAME_oneline(
wolfSSL_X509_get_subject_name(peer), 0, 0); wolfSSL_X509_get_subject_name(peer), 0, 0);
printf("peer's cert info:\n issuer : %s\n subject: %s\n", issuer, printf("\tPeer's cert info:\n issuer : %s\n subject: %s\n", issuer,
subject); subject);
XFREE(subject, 0, DYNAMIC_TYPE_OPENSSL); XFREE(subject, 0, DYNAMIC_TYPE_OPENSSL);
XFREE(issuer, 0, DYNAMIC_TYPE_OPENSSL); XFREE(issuer, 0, DYNAMIC_TYPE_OPENSSL);
} }
else else
printf("peer has no cert!\n"); printf("\tPeer has no cert!\n");
#else
printf("\tPeer certs: %d\n", store->totalCerts);
#endif #endif
printf("Subject's domain name is %s\n", store->domain);
printf("Allowing to continue anyway (shouldn't do this, EVER!!!)\n"); printf("\tSubject's domain name is %s\n", store->domain);
printf("\tAllowing to continue anyway (shouldn't do this, EVER!!!)\n");
return 1; return 1;
} }
@@ -1267,7 +1270,7 @@ static INLINE void CaCb(unsigned char* der, int sz, int type)
static INLINE int ChangeToWolfRoot(void) static INLINE int ChangeToWolfRoot(void)
{ {
#if !defined(NO_FILESYSTEM) #if !defined(NO_FILESYSTEM)
int depth, res; int depth, res;
XFILE file; XFILE file;
for(depth = 0; depth <= MAX_WOLF_ROOT_DEPTH; depth++) { for(depth = 0; depth <= MAX_WOLF_ROOT_DEPTH; depth++) {
@@ -1286,7 +1289,7 @@ static INLINE void CaCb(unsigned char* der, int sz, int type)
break; break;
} }
} }
err_sys("wolf root not found"); err_sys("wolf root not found");
return -1; return -1;
#else #else