Merge remote-tracking branch 'upstream-public/pr/917' into mbedtls-1.3

This commit is contained in:
Gilles Peskine 2017-11-29 20:55:03 +01:00
commit 9f423b18cb
2 changed files with 8 additions and 1 deletions

View File

@ -31,6 +31,8 @@ Bugfix
* Don't print X.509 version tag for v1 CRT's, and omit extensions for * Don't print X.509 version tag for v1 CRT's, and omit extensions for
non-v3 CRT's. non-v3 CRT's.
* Fix bugs in RSA test suite under MBEDTLS_NO_PLATFORM_ENTROPY. #1023 #1024 * Fix bugs in RSA test suite under MBEDTLS_NO_PLATFORM_ENTROPY. #1023 #1024
* Fix net_would_block to avoid modification by errno through fcntl call.
Found by nkolban. Fixes #845.
Changes Changes
* Extend cert_write example program by options to set the CRT version * Extend cert_write example program by options to set the CRT version

View File

@ -404,13 +404,18 @@ static int net_would_block( int fd )
*/ */
static int net_would_block( int fd ) static int net_would_block( int fd )
{ {
int err = errno;
/* /*
* Never return 'WOULD BLOCK' on a non-blocking socket * Never return 'WOULD BLOCK' on a non-blocking socket
*/ */
if( ( fcntl( fd, F_GETFL ) & O_NONBLOCK ) != O_NONBLOCK ) if( ( fcntl( fd, F_GETFL ) & O_NONBLOCK ) != O_NONBLOCK )
{
errno = err;
return( 0 ); return( 0 );
}
switch( errno ) switch( errno = err )
{ {
#if defined EAGAIN #if defined EAGAIN
case EAGAIN: case EAGAIN: