ports/libssh
ports/libssh
1
1
Форкнуть 0
Код Релизы 1 Активность

libgcrypt: fix cipher handle leaks on setkey error paths

Просмотр исходного кода 
Signed-off-by: Jussi Kivilinna <jussi.kivilinna@iki.fi>
Reviewed-by: Jakub Jelen <jjelen@redhat.com>
Reviewed-by: Andreas Schneider <asn@cryptomilk.org>
Этот коммит содержится в:
Jussi Kivilinna 2019-11-25 23:21:35 +02:00
родитель de4b8f88a2
Коммит 08f4469e21
1 изменённых файлов: 7 добавлений и 1 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

8
src/libgcrypt.c
Просмотреть файл

@ -302,10 +302,12 @@ static int blowfish_set_key(struct ssh_cipher_struct *cipher, void *key, void *I
return -1; return -1;
} }
if (gcry_cipher_setkey(cipher->key[0], key, 16)) { if (gcry_cipher_setkey(cipher->key[0], key, 16)) {
gcry_cipher_close(cipher->key[0]);
SAFE_FREE(cipher->key); SAFE_FREE(cipher->key);
return -1; return -1;
} }
if (gcry_cipher_setiv(cipher->key[0], IV, 8)) { if (gcry_cipher_setiv(cipher->key[0], IV, 8)) {
gcry_cipher_close(cipher->key[0]);
SAFE_FREE(cipher->key); SAFE_FREE(cipher->key);
return -1; return -1;
} }
@ -359,12 +361,13 @@ static int aes_set_key(struct ssh_cipher_struct *cipher, void *key, void *IV) {
break; break;
} }
if (gcry_cipher_setkey(cipher->key[0], key, cipher->keysize / 8)) { if (gcry_cipher_setkey(cipher->key[0], key, cipher->keysize / 8)) {
gcry_cipher_close(cipher->key[0]);
SAFE_FREE(cipher->key); SAFE_FREE(cipher->key);
return -1; return -1;
} }
if(mode == GCRY_CIPHER_MODE_CBC){ if(mode == GCRY_CIPHER_MODE_CBC){
if (gcry_cipher_setiv(cipher->key[0], IV, 16)) { if (gcry_cipher_setiv(cipher->key[0], IV, 16)) {
gcry_cipher_close(cipher->key[0]);
SAFE_FREE(cipher->key); SAFE_FREE(cipher->key);
return -1; return -1;
} }
@ -375,6 +378,7 @@ static int aes_set_key(struct ssh_cipher_struct *cipher, void *key, void *IV) {
memcpy(cipher->last_iv, IV, AES_GCM_IVLEN); memcpy(cipher->last_iv, IV, AES_GCM_IVLEN);
} else { } else {
if(gcry_cipher_setctr(cipher->key[0],IV,16)){ if(gcry_cipher_setctr(cipher->key[0],IV,16)){
gcry_cipher_close(cipher->key[0]);
SAFE_FREE(cipher->key); SAFE_FREE(cipher->key);
return -1; return -1;
} }
@ -565,10 +569,12 @@ static int des3_set_key(struct ssh_cipher_struct *cipher, void *key, void *IV) {
return -1; return -1;
} }
if (gcry_cipher_setkey(cipher->key[0], key, 24)) { if (gcry_cipher_setkey(cipher->key[0], key, 24)) {
gcry_cipher_close(cipher->key[0]);
SAFE_FREE(cipher->key); SAFE_FREE(cipher->key);
return -1; return -1;
} }
if (gcry_cipher_setiv(cipher->key[0], IV, 8)) { if (gcry_cipher_setiv(cipher->key[0], IV, 8)) {
gcry_cipher_close(cipher->key[0]);
SAFE_FREE(cipher->key); SAFE_FREE(cipher->key);
return -1; return -1;
} }