<p>In plugins/sslutils.c:</p>
<pre style='color:#555'>> @@ -176,10 +177,15 @@ int np_net_ssl_check_cert(int days_till_exp_warn, int days_till_exp_crit){
>    /* Retrieve timestamp of certificate */
>    tm = X509_get_notAfter(certificate);
>  
> +  /* Retrieve subject name in certificate. */
> +  *subject = '\0';
> +  X509_NAME_get_text_by_NID(X509_get_subject_name(certificate), NID_commonName, subject, 256);
</pre>
<p>shouldn't be the subject already stored in 'subj'? see <a href="https://github.com/monitoring-plugins/monitoring-plugins/blob/master/plugins/sslutils.c#L166">https://github.com/monitoring-plugins/monitoring-plugins/blob/master/plugins/sslutils.c#L166</a><br>
So we don't need to gather that once again.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br><a href="https://github.com/monitoring-plugins/monitoring-plugins/pull/1174/files#r15516933">Reply to this email on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/5514713__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcyMjI1MDkyMywiZGF0YSI6eyJpZCI6MTc2MjQ5NTV9fQ==--3d272f3fb2a524576d709b94515e864c31c45425.gif" width="1" /></p>