Collapse two if blocks into one

[privoxy.git] / ssl.c
diff --git a/ssl.c b/ssl.c

index 25f86e4..85701ce 100644 (file)
--- a/ssl.c
+++ b/ssl.c
@@ -28,6 +28,7 @@
   *
   *********************************************************************/
  
+#include <ctype.h>
  #include <string.h>
  #include <unistd.h>
  
@@ -1519,6 +1520,50 @@ exit:
  
  }
  
+
+/*********************************************************************
+ *
+ * Function    :  host_is_ip_address
+ *
+ * Description :  Checks whether or not a host is specified by
+ *                IP address. Does not actually validate the
+ *                address.
+ *
+ * Parameters  :
+ *          1  :  host = The host name to check
+ *
+ * Returns     :   1 => Yes
+ *                 0 => No
+ *
+ *********************************************************************/
+static int host_is_ip_address(const char *host)
+{
+   const char *p;
+
+   if (NULL != strstr(host, ":"))
+   {
+      /* Assume an IPv6 address. */
+      return 1;
+   }
+
+   for (p = host; *p; p++)
+   {
+      if ((*p != '.') && !privoxy_isdigit(*p))
+      {
+         /* Not a dot or digit so it can't be an IPv4 address. */
+         return 0;
+      }
+   }
+
+   /*
+    * Host only consists of dots and digits so
+    * assume that is an IPv4 address.
+    */
+   return 1;
+
+}
+
+
  /*********************************************************************
   *
   * Function    :  generate_webpage_certificate
@@ -1894,7 +1939,8 @@ static int generate_webpage_certificate(struct client_state *csp)
     }
  #endif /* MBEDTLS_SHA1_C */
  
-   if (set_subject_alternative_name(&cert, csp->http->host))
+   if (!host_is_ip_address(csp->http->host) &&
+      set_subject_alternative_name(&cert, csp->http->host))
     {
        /* Errors are already logged by set_subject_alternative_name() */
        ret = -1;
@@ -2064,6 +2110,7 @@ extern void ssl_send_certificate_error(struct client_state *csp)
        "HTTP/1.1 200 OK\r\n"
        "Content-Type: text/html\r\n"
        "Connection: close\r\n\r\n"
+      "<!DOCTYPE html>\n"
        "<html><head><title>Server certificate verification failed</title></head>\n"
        "<body><h1>Server certificate verification failed</h1>\n"
        "<p><a href=\"https://" CGI_SITE_2_HOST "/\">Privoxy</a> was unable "