Remove redundant `else` statement
Remove `else` statement, as it is redundant. resolves #1776
diff --git a/ChangeLog b/ChangeLog
index abd5e61..0d3d40a 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -10,6 +10,9 @@
* Add ecc extensions only if an ecc based ciphersuite is used.
This improves compliance to RFC 4492, and as a result, solves
interoperability issues with BouncyCastle. Raised by milenamil in #1157.
+ * Remove redundant else statement, which is not readable, and the positive
+ path in the if statement results in exiting the funciton. Raised by irwir
+ in #1776.
Changes
* Copy headers preserving timestamps when doing a "make install".
diff --git a/library/ecp.c b/library/ecp.c
index 68c6f49..9e2c085 100644
--- a/library/ecp.c
+++ b/library/ecp.c
@@ -1897,7 +1897,6 @@
mbedtls_mpi_get_bit( d, 1 ) != 0 ||
mbedtls_mpi_bitlen( d ) - 1 != grp->nbits ) /* mbedtls_mpi_bitlen is one-based! */
return( MBEDTLS_ERR_ECP_INVALID_KEY );
- else
/* see [Curve25519] page 5 */
if( grp->nbits == 254 && mbedtls_mpi_get_bit( d, 2 ) != 0 )