wolfSSL/wolfssl
1
0
Fork 1
mirror of https://github.com/wolfSSL/wolfssl.git synced 2025-07-30 18:57:27 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2795 from SparkiDev/tls13_secret_cb

Browse Source 
Call secret callback when TLS 1.3 secrets generated
This commit is contained in:
toddouska
2020-02-17 11:41:16 -08:00
committed by GitHub
parent fda322829f 614e675a00
commit 44c327ee14
5 changed files with 148 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/internal.c
View File

@ -5501,6 +5501,10 @@ int InitSSL(WOLFSSL* ssl, WOLFSSL_CTX* ctx, int writeDup)
#ifdef HAVE_SECRET_CALLBACK #ifdef HAVE_SECRET_CALLBACK
ssl->sessionSecretCb = NULL; ssl->sessionSecretCb = NULL;
ssl->sessionSecretCtx = NULL; ssl->sessionSecretCtx = NULL;
#ifdef WOLFSSL_TLS13
ssl->tls13SecretCb = NULL;
ssl->tls13SecretCtx = NULL;
#endif
#endif #endif
#ifdef HAVE_SESSION_TICKET #ifdef HAVE_SESSION_TICKET
@ -17414,6 +17418,9 @@ const char* wolfSSL_ERR_reason_error_string(unsigned long e)
case SSL_SHUTDOWN_ALREADY_DONE_E: case SSL_SHUTDOWN_ALREADY_DONE_E:
return "Shutdown has already occurred"; return "Shutdown has already occurred";
case TLS13_SECRET_CB_E:
return "TLS1.3 Secret Callback Error";
default : default :
return "unknown error number"; return "unknown error number";
} }

137
src/tls13.c
View File

@ -440,7 +440,6 @@ static int DeriveKey(WOLFSSL* ssl, byte* output, int outputLen,
hash, hashOutSz, digestAlg); hash, hashOutSz, digestAlg);
} }
#ifndef NO_PSK #ifndef NO_PSK
#ifdef WOLFSSL_TLS13_DRAFT_18 #ifdef WOLFSSL_TLS13_DRAFT_18
/* The length of the binder key label. */ /* The length of the binder key label. */
@ -521,10 +520,21 @@ static const byte earlyTrafficLabel[EARLY_TRAFFIC_LABEL_SZ + 1] =
*/ */
static int DeriveEarlyTrafficSecret(WOLFSSL* ssl, byte* key) static int DeriveEarlyTrafficSecret(WOLFSSL* ssl, byte* key)
{ {
int ret;
WOLFSSL_MSG("Derive Early Traffic Secret"); WOLFSSL_MSG("Derive Early Traffic Secret");
return DeriveKey(ssl, key, -1, ssl->arrays->secret, ret = DeriveKey(ssl, key, -1, ssl->arrays->secret,
earlyTrafficLabel, EARLY_TRAFFIC_LABEL_SZ, earlyTrafficLabel, EARLY_TRAFFIC_LABEL_SZ,
ssl->specs.mac_algorithm, 1); ssl->specs.mac_algorithm, 1);
#ifdef HAVE_SECRET_CALLBACK
if (ret == 0 && ssl->tls13SecretCb != NULL) {
ret = ssl->tls13SecretCb(ssl, CLIENT_EARLY_TRAFFIC_SECRET, key,
ssl->specs.hash_size, ssl->tls13SecretCtx);
if (ret != 0) {
return TLS13_SECRET_CB_E;
}
}
#endif /* HAVE_SECRET_CALLBACK */
return ret;
} }
#ifdef TLS13_SUPPORTS_EXPORTERS #ifdef TLS13_SUPPORTS_EXPORTERS
@ -549,10 +559,21 @@ static const byte earlyExporterLabel[EARLY_EXPORTER_LABEL_SZ + 1] =
*/ */
static int DeriveEarlyExporterSecret(WOLFSSL* ssl, byte* key) static int DeriveEarlyExporterSecret(WOLFSSL* ssl, byte* key)
{ {
int ret;
WOLFSSL_MSG("Derive Early Exporter Secret"); WOLFSSL_MSG("Derive Early Exporter Secret");
return DeriveKey(ssl, key, -1, ssl->arrays->secret, ret = DeriveKey(ssl, key, -1, ssl->arrays->secret,
earlyExporterLabel, EARLY_EXPORTER_LABEL_SZ, earlyExporterLabel, EARLY_EXPORTER_LABEL_SZ,
ssl->specs.mac_algorithm, 1); ssl->specs.mac_algorithm, 1);
#ifdef HAVE_SECRET_CALLBACK
if (ret == 0 && ssl->tls13SecretCb != NULL) {
ret = ssl->tls13SecretCb(ssl, EARLY_EXPORTER_SECRET, key
ssl->specs.hash_size, ssl->tls13SecretCtx);
if (ret != 0) {
return TLS13_SECRET_CB_E;
}
}
#endif /* HAVE_SECRET_CALLBACK */
return ret;
} }
#endif #endif
#endif #endif
@ -578,10 +599,21 @@ static const byte clientHandshakeLabel[CLIENT_HANDSHAKE_LABEL_SZ + 1] =
*/ */
static int DeriveClientHandshakeSecret(WOLFSSL* ssl, byte* key) static int DeriveClientHandshakeSecret(WOLFSSL* ssl, byte* key)
{ {
int ret;
WOLFSSL_MSG("Derive Client Handshake Secret"); WOLFSSL_MSG("Derive Client Handshake Secret");
return DeriveKey(ssl, key, -1, ssl->arrays->preMasterSecret, ret = DeriveKey(ssl, key, -1, ssl->arrays->preMasterSecret,
clientHandshakeLabel, CLIENT_HANDSHAKE_LABEL_SZ, clientHandshakeLabel, CLIENT_HANDSHAKE_LABEL_SZ,
ssl->specs.mac_algorithm, 1); ssl->specs.mac_algorithm, 1);
#ifdef HAVE_SECRET_CALLBACK
if (ret == 0 && ssl->tls13SecretCb != NULL) {
ret = ssl->tls13SecretCb(ssl, CLIENT_HANDSHAKE_TRAFFIC_SECRET, key,
ssl->specs.hash_size, ssl->tls13SecretCtx);
if (ret != 0) {
return TLS13_SECRET_CB_E;
}
}
#endif /* HAVE_SECRET_CALLBACK */
return ret;
} }
#ifdef WOLFSSL_TLS13_DRAFT_18 #ifdef WOLFSSL_TLS13_DRAFT_18
@ -605,10 +637,21 @@ static const byte serverHandshakeLabel[SERVER_HANDSHAKE_LABEL_SZ + 1] =
*/ */
static int DeriveServerHandshakeSecret(WOLFSSL* ssl, byte* key) static int DeriveServerHandshakeSecret(WOLFSSL* ssl, byte* key)
{ {
int ret;
WOLFSSL_MSG("Derive Server Handshake Secret"); WOLFSSL_MSG("Derive Server Handshake Secret");
return DeriveKey(ssl, key, -1, ssl->arrays->preMasterSecret, ret = DeriveKey(ssl, key, -1, ssl->arrays->preMasterSecret,
serverHandshakeLabel, SERVER_HANDSHAKE_LABEL_SZ, serverHandshakeLabel, SERVER_HANDSHAKE_LABEL_SZ,
ssl->specs.mac_algorithm, 1); ssl->specs.mac_algorithm, 1);
#ifdef HAVE_SECRET_CALLBACK
if (ret == 0 && ssl->tls13SecretCb != NULL) {
ret = ssl->tls13SecretCb(ssl, SERVER_HANDSHAKE_TRAFFIC_SECRET, key,
ssl->specs.hash_size, ssl->tls13SecretCtx);
if (ret != 0) {
return TLS13_SECRET_CB_E;
}
}
#endif /* HAVE_SECRET_CALLBACK */
return ret;
} }
#ifdef WOLFSSL_TLS13_DRAFT_18 #ifdef WOLFSSL_TLS13_DRAFT_18
@ -632,10 +675,21 @@ static const byte clientAppLabel[CLIENT_APP_LABEL_SZ + 1] =
*/ */
static int DeriveClientTrafficSecret(WOLFSSL* ssl, byte* key) static int DeriveClientTrafficSecret(WOLFSSL* ssl, byte* key)
{ {
int ret;
WOLFSSL_MSG("Derive Client Traffic Secret"); WOLFSSL_MSG("Derive Client Traffic Secret");
return DeriveKey(ssl, key, -1, ssl->arrays->masterSecret, ret = DeriveKey(ssl, key, -1, ssl->arrays->masterSecret,
clientAppLabel, CLIENT_APP_LABEL_SZ, clientAppLabel, CLIENT_APP_LABEL_SZ,
ssl->specs.mac_algorithm, 1); ssl->specs.mac_algorithm, 1);
#ifdef HAVE_SECRET_CALLBACK
if (ret == 0 && ssl->tls13SecretCb != NULL) {
ret = ssl->tls13SecretCb(ssl, CLIENT_TRAFFIC_SECRET, key,
ssl->specs.hash_size, ssl->tls13SecretCtx);
if (ret != 0) {
return TLS13_SECRET_CB_E;
}
}
#endif /* HAVE_SECRET_CALLBACK */
return ret;
} }
#ifdef WOLFSSL_TLS13_DRAFT_18 #ifdef WOLFSSL_TLS13_DRAFT_18
@ -659,10 +713,21 @@ static const byte serverAppLabel[SERVER_APP_LABEL_SZ + 1] =
*/ */
static int DeriveServerTrafficSecret(WOLFSSL* ssl, byte* key) static int DeriveServerTrafficSecret(WOLFSSL* ssl, byte* key)
{ {
int ret;
WOLFSSL_MSG("Derive Server Traffic Secret"); WOLFSSL_MSG("Derive Server Traffic Secret");
return DeriveKey(ssl, key, -1, ssl->arrays->masterSecret, ret = DeriveKey(ssl, key, -1, ssl->arrays->masterSecret,
serverAppLabel, SERVER_APP_LABEL_SZ, serverAppLabel, SERVER_APP_LABEL_SZ,
ssl->specs.mac_algorithm, 1); ssl->specs.mac_algorithm, 1);
#ifdef HAVE_SECRET_CALLBACK
if (ret == 0 && ssl->tls13SecretCb != NULL) {
ret = ssl->tls13SecretCb(ssl, SERVER_TRAFFIC_SECRET, key,
ssl->specs.hash_size, ssl->tls13SecretCtx);
if (ret != 0) {
return TLS13_SECRET_CB_E;
}
}
#endif /* HAVE_SECRET_CALLBACK */
return ret;
} }
#ifdef TLS13_SUPPORTS_EXPORTERS #ifdef TLS13_SUPPORTS_EXPORTERS
@ -687,10 +752,21 @@ static const byte exporterMasterLabel[EXPORTER_MASTER_LABEL_SZ + 1] =
*/ */
static int DeriveExporterSecret(WOLFSSL* ssl, byte* key) static int DeriveExporterSecret(WOLFSSL* ssl, byte* key)
{ {
int ret;
WOLFSSL_MSG("Derive Exporter Secret"); WOLFSSL_MSG("Derive Exporter Secret");
return DeriveKey(ssl, key, -1, ssl->arrays->masterSecret, ret = DeriveKey(ssl, key, -1, ssl->arrays->masterSecret,
exporterMasterLabel, EXPORTER_MASTER_LABEL_SZ, exporterMasterLabel, EXPORTER_MASTER_LABEL_SZ,
ssl->specs.mac_algorithm, 1); ssl->specs.mac_algorithm, 1);
#ifdef HAVE_SECRET_CALLBACK
if (ret == 0 && ssl->tls13SecretCb != NULL) {
ret = ssl->tls13SecretCb(ssl, EXPORTER_SECRET, key,
ssl->specs.hash_size, ssl->tls13SecretCtx);
if (ret != 0) {
return TLS13_SECRET_CB_E;
}
}
#endif /* HAVE_SECRET_CALLBACK */
return ret;
} }
#endif #endif
@ -3087,8 +3163,9 @@ int DoTls13ServerHello(WOLFSSL* ssl, const byte* input, word32* inOutIdx,
int secretSz = SECRET_LEN; int secretSz = SECRET_LEN;
ret = ssl->sessionSecretCb(ssl, ssl->session.masterSecret, ret = ssl->sessionSecretCb(ssl, ssl->session.masterSecret,
&secretSz, ssl->sessionSecretCtx); &secretSz, ssl->sessionSecretCtx);
if (ret != 0 || secretSz != SECRET_LEN) if (ret != 0 || secretSz != SECRET_LEN) {
return SESSION_SECRET_CB_E; return SESSION_SECRET_CB_E;
}
} }
#endif /* HAVE_SECRET_CALLBACK */ #endif /* HAVE_SECRET_CALLBACK */
@ -8844,6 +8921,20 @@ int wolfSSL_read_early_data(WOLFSSL* ssl, void* data, int sz, int* outSz)
} }
#endif #endif
#ifdef HAVE_SECRET_CALLBACK
int wolfSSL_set_tls13_secret_cb(WOLFSSL* ssl, Tls13SecretCb cb, void* ctx)
{
WOLFSSL_ENTER("wolfSSL_set_tls13_secret_cb");
if (ssl == NULL)
return WOLFSSL_FATAL_ERROR;
ssl->tls13SecretCb = cb;
ssl->tls13SecretCtx = ctx;
return WOLFSSL_SUCCESS;
}
#endif
#undef ERROR_OUT #undef ERROR_OUT
#endif /* !WOLFCRYPT_ONLY */ #endif /* !WOLFCRYPT_ONLY */

1
wolfssl/error-ssl.h
View File

@ -166,6 +166,7 @@ enum wolfSSL_ErrorCodes {
TSIP_MAC_DIGSZ_E = -435, /* Invalid MAC size for TSIP */ TSIP_MAC_DIGSZ_E = -435, /* Invalid MAC size for TSIP */
CLIENT_CERT_CB_ERROR = -436, /* Client cert callback error */ CLIENT_CERT_CB_ERROR = -436, /* Client cert callback error */
SSL_SHUTDOWN_ALREADY_DONE_E = -437, /* Shutdown called redundantly */ SSL_SHUTDOWN_ALREADY_DONE_E = -437, /* Shutdown called redundantly */
TLS13_SECRET_CB_E = -438, /* TLS1.3 secret Cb fcn failure */
/* add strings to wolfSSL_ERR_reason_error_string in internal.c !!!!! */ /* add strings to wolfSSL_ERR_reason_error_string in internal.c !!!!! */

4
wolfssl/internal.h
View File

@ -4107,6 +4107,10 @@ struct WOLFSSL {
#ifdef HAVE_SECRET_CALLBACK #ifdef HAVE_SECRET_CALLBACK
SessionSecretCb sessionSecretCb; SessionSecretCb sessionSecretCb;
void* sessionSecretCtx; void* sessionSecretCtx;
#ifdef WOLFSSL_TLS13
Tls13SecretCb tls13SecretCb;
void* tls13SecretCtx;
#endif
#endif /* HAVE_SECRET_CALLBACK */ #endif /* HAVE_SECRET_CALLBACK */
#ifdef WOLFSSL_JNI #ifdef WOLFSSL_JNI
void* jObjectRef; /* reference to WolfSSLSession in JNI wrapper */ void* jObjectRef; /* reference to WolfSSLSession in JNI wrapper */

25
wolfssl/ssl.h
View File

@ -643,6 +643,19 @@ enum AlertLevel {
/* Maximum number of groups that can be set */ /* Maximum number of groups that can be set */
#define WOLFSSL_MAX_GROUP_COUNT 10 #define WOLFSSL_MAX_GROUP_COUNT 10
#if defined(HAVE_SECRET_CALLBACK) && defined(WOLFSSL_TLS13)
enum Tls13Secret {
CLIENT_EARLY_TRAFFIC_SECRET,
CLIENT_HANDSHAKE_TRAFFIC_SECRET,
SERVER_HANDSHAKE_TRAFFIC_SECRET,
CLIENT_TRAFFIC_SECRET,
SERVER_TRAFFIC_SECRET,
EARLY_EXPORTER_SECRET,
EXPORTER_SECRET
};
#endif
typedef WOLFSSL_METHOD* (*wolfSSL_method_func)(void* heap); typedef WOLFSSL_METHOD* (*wolfSSL_method_func)(void* heap);
/* CTX Method EX Constructor Functions */ /* CTX Method EX Constructor Functions */
@ -957,9 +970,15 @@ WOLFSSL_ABI WOLFSSL_API long wolfSSL_CTX_set_session_cache_mode(WOLFSSL_CTX*,
long); long);
#ifdef HAVE_SECRET_CALLBACK #ifdef HAVE_SECRET_CALLBACK
typedef int (*SessionSecretCb)(WOLFSSL* ssl, typedef int (*SessionSecretCb)(WOLFSSL* ssl, void* secret, int* secretSz,
void* secret, int* secretSz, void* ctx); void* ctx);
WOLFSSL_API int wolfSSL_set_session_secret_cb(WOLFSSL*, SessionSecretCb, void*); WOLFSSL_API int wolfSSL_set_session_secret_cb(WOLFSSL*, SessionSecretCb,
void*);
#ifdef WOLFSSL_TLS13
typedef int (*Tls13SecretCb)(WOLFSSL* ssl, int id, const unsigned char* secret,
int secretSz, void* ctx);
WOLFSSL_API int wolfSSL_set_tls13_secret_cb(WOLFSSL*, Tls13SecretCb, void*);
#endif
#endif /* HAVE_SECRET_CALLBACK */ #endif /* HAVE_SECRET_CALLBACK */
/* session cache persistence */ /* session cache persistence */