yuzu/mbedtls
1
0
Fork 0
mirror of https://github.com/yuzu-emu/mbedtls.git synced 2024-12-23 09:35:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix thread-safety issue in debug.c

Browse source 
Closes #203
This commit is contained in:
Manuel Pégourié-Gonnard 2015-06-29 14:57:45 +02:00
parent 8e8ae3d961
commit 6c3ccf5fd0
4 changed files with 31 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
ChangeLog
View file

@ -1,5 +1,10 @@
mbed TLS ChangeLog (Sorted per branch, date)
= mbed TLS 1.3.12 released 2015-07-??
Bugfix
* Fix thread-safety issue in SSL debug module (found by Edwin van Vliet).
= mbed TLS 1.3.11 released 2015-06-04
Security

5
include/polarssl/debug.h
View file

@ -57,7 +57,7 @@
#define SSL_DEBUG_MSG( level, args ) \
debug_print_msg( ssl, level, __FILE__, __LINE__, debug_fmt args );
debug_print_msg_free( ssl, level, __FILE__, __LINE__, debug_fmt args );
#define SSL_DEBUG_RET( level, text, ret ) \
debug_print_ret( ssl, level, __FILE__, __LINE__, text, ret );
@ -115,6 +115,9 @@ void debug_set_threshold( int threshold );
char *debug_fmt( const char *format, ... );
void debug_print_msg_free( const ssl_context *ssl, int level,
const char *file, int line, char *text );
void debug_print_msg( const ssl_context *ssl, int level,
const char *file, int line, const char *text );

25
library/debug.c
View file

@ -47,9 +47,13 @@
#if defined(POLARSSL_PLATFORM_C)
#include "polarssl/platform.h"
#else
#define polarssl_snprintf snprintf
#define polarssl_snprintf snprintf
#define polarssl_malloc malloc
#define polarssl_free free
#endif
#define DEBUG_BUF_SIZE 512
static int debug_log_mode = POLARSSL_DEBUG_DFL_MODE;
static int debug_threshold = 0;
@ -66,17 +70,28 @@ void debug_set_threshold( int threshold )
char *debug_fmt( const char *format, ... )
{
va_list argp;
static char str[512];
int maxlen = sizeof( str ) - 1;
char *str = polarssl_malloc( DEBUG_BUF_SIZE );
if( str == NULL )
return;
va_start( argp, format );
vsnprintf( str, maxlen, format, argp );
vsnprintf( str, DEBUG_BUF_SIZE - 1, format, argp );
va_end( argp );
str[maxlen] = '\0';
str[DEBUG_BUF_SIZE - 1] = '\0';
return( str );
}
void debug_print_msg_free( const ssl_context *ssl, int level,
const char *file, int line, char *text )
{
if( text != NULL )
debug_print_msg( ssl, level, file, line, text );
polarssl_free( text );
}
void debug_print_msg( const ssl_context *ssl, int level,
const char *file, int line, const char *text )
{

4
tests/suites/test_suite_debug.function
View file

@ -44,8 +44,8 @@ void debug_print_msg_threshold( int threshold, int level, char *file, int line,
debug_set_threshold( threshold );
ssl_set_dbg(&ssl, string_debug, &buffer);
debug_print_msg( &ssl, level, file, line,
debug_fmt("Text message, 2 == %d", 2 ) );
debug_print_msg_free( &ssl, level, file, line,
debug_fmt("Text message, 2 == %d", 2 ) );
TEST_ASSERT( strcmp( buffer.buf, result_str ) == 0 );
}