diff options
| author | Luca Deri <deri@ntop.org> | 2020-08-05 17:57:00 +0200 |
|---|---|---|
| committer | Luca Deri <deri@ntop.org> | 2020-08-05 17:57:00 +0200 |
| commit | d3fb1fb25a2b3ab37d4e1964f4fdde70d24349e1 (patch) | |
| tree | bf264e004c15560cf3cf49c72cbda7ba4d87becf /src | |
| parent | 07d9fa7f96d50aea4a1d8ed40330afa7d4944151 (diff) | |
Fixed possible memory leak in TLS certificate handling
Diffstat (limited to 'src')
| -rw-r--r-- | src/lib/protocols/tls.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/lib/protocols/tls.c b/src/lib/protocols/tls.c index ec267ba5e..5642ebdf0 100644 --- a/src/lib/protocols/tls.c +++ b/src/lib/protocols/tls.c @@ -316,7 +316,9 @@ static void processCertificateElements(struct ndpi_detection_module_struct *ndpi printf("[TLS] %s() IssuerDN [%s]\n", __FUNCTION__, rdnSeqBuf); #endif - if(rdn_len) flow->protos.stun_ssl.ssl.issuerDN = ndpi_strdup(rdnSeqBuf); + if(rdn_len && (flow->protos.stun_ssl.ssl.issuerDN == NULL)) + flow->protos.stun_ssl.ssl.issuerDN = ndpi_strdup(rdnSeqBuf); + rdn_len = 0; /* Reset buffer */ } |