Skip to content

Commit 76064e8

Browse files
author
Ben Taylor
committed
tidied up error handling
Signed-off-by: Ben Taylor <ben.taylor@linaro.org>
1 parent d222ff7 commit 76064e8

1 file changed

Lines changed: 2 additions & 6 deletions

File tree

drivers/builtin/src/asn1write.c

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -453,12 +453,8 @@ int mbedtls_asn1_write_integer(unsigned char **p, unsigned char *start, const un
453453
return MBEDTLS_ERR_ASN1_INVALID_DATA; //TC1 NULL Pointer exceptions.
454454
}
455455

456-
if(input_buffer_size<3){
457-
return MBEDTLS_ERR_ASN1_BUF_TOO_SMALL;//TC2 buffer too small for 1 byte integer and header.
458-
}
459-
460-
if(input_buffer_size<integer_length){
461-
return MBEDTLS_ERR_ASN1_BUF_TOO_SMALL;//TC3 buffer less than integer size.
456+
if((input_buffer_size<integer_length) || (input_buffer_size<3)){
457+
return MBEDTLS_ERR_ASN1_BUF_TOO_SMALL;//TC3 buffer less than integer size. TC2 buffer too small for 1 byte integer and header.
462458
}
463459

464460
memset(start, 0, input_buffer_size);

0 commit comments

Comments
 (0)