Fix check_smtp and check_tcp where duplicate messages were displayed for certificate errors (Opsview team)

3 files changed, 2 insertions, 7 deletions

diff --git a/NEWS b/NEWS
index 9ff87554..fd9b8e84 100644
--- a/NEWS
+++ b/NEWS
@@ -16,6 +16,7 @@ This file documents the major additions and syntax changes between releases.
         Fix check_ldap overriding the port when --ssl was specified after -p
         Fix check_procs where regex input of '|' would get displayed in output - now replaced with ','
         Fix segfault in check_host when hostname returns multiple IP addresses (Sebastian Harl)
+        Fix check_smtp and check_tcp where duplicate messages were displayed for certificate errors
 
 1.4.15 27th July 2010
         ENHANCEMENTS
diff --git a/plugins/check_smtp.c b/plugins/check_smtp.c
index 6c5994c0..494bc2cd 100644
--- a/plugins/check_smtp.c
+++ b/plugins/check_smtp.c
@@ -276,9 +276,6 @@ main (int argc, char **argv)
 #  ifdef USE_OPENSSL
                   if ( check_cert ) {
                     result = np_net_ssl_check_cert(days_till_exp);
-                    if(result != STATE_OK){
-                      printf ("%s\n", _("CRITICAL - Cannot retrieve server certificate."));
-                    }
                     my_close();
                     return result;
                   }
diff --git a/plugins/check_tcp.c b/plugins/check_tcp.c
index 178bd568..d3c92a49 100644
--- a/plugins/check_tcp.c
+++ b/plugins/check_tcp.c
@@ -236,12 +236,9 @@ main (int argc, char **argv)
                 result = np_net_ssl_init(sd);
                 if (result == STATE_OK && check_cert == TRUE) {
                         result = np_net_ssl_check_cert(days_till_exp);
-                        if(result != STATE_OK) {
-                                printf(_("CRITICAL - Cannot retrieve server certificate.\n"));
-                        }
                 }
         }
-        if(result != STATE_OK){
+        if(result != STATE_OK || check_cert == TRUE){
                 np_net_ssl_cleanup();
                 if(sd) close(sd);
                 return result;