kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-10-02 10:22:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

res_rtp_asterisk.c: Fix off nominal memory leak.

Browse Source 
Change-Id: I95b1088d11244a2edae6607c12fbf33b38658a75
This commit is contained in:
Richard Mudgett
2016-12-07 15:23:02 -06:00
parent bc6fc3ab4d
commit e2fa3c7eda
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
res/res_rtp_asterisk.c
View File

@@ -1455,12 +1455,6 @@ static int ast_rtp_dtls_set_configuration(struct ast_rtp_instance *instance, con
return -1; return -1;
} }
if (!(certbio = BIO_new(BIO_s_file()))) {
ast_log(LOG_ERROR, "Failed to allocate memory for certificate fingerprinting on RTP instance '%p'\n",
instance);
return -1;
}
if (rtp->local_hash == AST_RTP_DTLS_HASH_SHA1) { if (rtp->local_hash == AST_RTP_DTLS_HASH_SHA1) {
type = EVP_sha1(); type = EVP_sha1();
} else if (rtp->local_hash == AST_RTP_DTLS_HASH_SHA256) { } else if (rtp->local_hash == AST_RTP_DTLS_HASH_SHA256) {
@@ -1471,6 +1465,12 @@ static int ast_rtp_dtls_set_configuration(struct ast_rtp_instance *instance, con
return -1; return -1;
} }
if (!(certbio = BIO_new(BIO_s_file()))) {
ast_log(LOG_ERROR, "Failed to allocate memory for certificate fingerprinting on RTP instance '%p'\n",
instance);
return -1;
}
if (!BIO_read_filename(certbio, dtls_cfg->certfile) || if (!BIO_read_filename(certbio, dtls_cfg->certfile) ||
!(cert = PEM_read_bio_X509(certbio, NULL, 0, NULL)) || !(cert = PEM_read_bio_X509(certbio, NULL, 0, NULL)) ||
!X509_digest(cert, type, fingerprint, &size) || !X509_digest(cert, type, fingerprint, &size) ||