]> git.meshlink.io Git - meshlink/commitdiff
ecdh & ecdsa: avoid some possible memory leaks in error conditions.
authorSven-Haegar Koch <haegar@sdinet.de>
Sat, 14 Apr 2012 00:28:43 +0000 (02:28 +0200)
committerGuus Sliepen <guus@tinc-vpn.org>
Sun, 15 Apr 2012 22:54:50 +0000 (00:54 +0200)
src/openssl/ecdh.c
src/openssl/ecdsa.c

index dd8f53f5042d3f88d9aab4f889e2d5112271424b..871f9fbd2f89a67400031e367217772c24c8f83e 100644 (file)
 
 bool ecdh_generate_public(ecdh_t *ecdh, void *pubkey) {
        *ecdh = EC_KEY_new_by_curve_name(NID_secp521r1);
+       if(!*ecdh) {
+               logger(DEBUG_ALWAYS, LOG_ERR, "Generating EC key_by_curve_name failed: %s", ERR_error_string(ERR_get_error(), NULL));
+               return false;
+       }
+
        if(!EC_KEY_generate_key(*ecdh)) {
+               EC_KEY_free(*ecdh);
+               *ecdh = NULL;
                logger(DEBUG_ALWAYS, LOG_ERR, "Generating EC key failed: %s", ERR_error_string(ERR_get_error(), NULL));
                return false;
        }
        
        const EC_POINT *point = EC_KEY_get0_public_key(*ecdh);
        if(!point) {
+               EC_KEY_free(*ecdh);
+               *ecdh = NULL;
                logger(DEBUG_ALWAYS, LOG_ERR, "Getting public key failed: %s", ERR_error_string(ERR_get_error(), NULL));
                return false;
        }
 
        size_t result = EC_POINT_point2oct(EC_KEY_get0_group(*ecdh), point, POINT_CONVERSION_COMPRESSED, pubkey, ECDH_SIZE, NULL);
        if(!result) {
+               EC_KEY_free(*ecdh);
+               *ecdh = NULL;
                logger(DEBUG_ALWAYS, LOG_ERR, "Converting EC_POINT to binary failed: %s", ERR_error_string(ERR_get_error(), NULL));
                return false;
        }
@@ -59,6 +70,7 @@ bool ecdh_compute_shared(ecdh_t *ecdh, const void *pubkey, void *shared) {
 
        int result = EC_POINT_oct2point(EC_KEY_get0_group(*ecdh), point, pubkey, ECDH_SIZE, NULL);
        if(!result) {
+               EC_POINT_free(point);
                logger(DEBUG_ALWAYS, LOG_ERR, "Converting binary to EC_POINT failed: %s", ERR_error_string(ERR_get_error(), NULL));
                return false;
        }
index a2c7deb42664302a4a28a3e674567b5ab22c8f4b..8b02501525ad0da30bcca14d8ef7bab03d80690a 100644 (file)
 //
 bool ecdsa_set_base64_public_key(ecdsa_t *ecdsa, const char *p) {
        *ecdsa = EC_KEY_new_by_curve_name(NID_secp521r1);
-
+       if(!*ecdsa) {
+               logger(DEBUG_ALWAYS, LOG_DEBUG, "EC_KEY_new_by_curve_name failed: %s", ERR_error_string(ERR_get_error(), NULL));
+               return false;
+       }
+       
        int len = strlen(p);
        unsigned char pubkey[len / 4 * 3 + 3];
        const unsigned char *ppubkey = pubkey;