X-Git-Url: https://ruderich.org/simon/gitweb/?a=blobdiff_plain;f=src%2Fverify.c;h=14c7127b7c39cd01ad0b9aa0082cb1396b35f6ca;hb=2cc876ebafc566267a6de49fe722257977967091;hp=0052ca61cdb73bb5d2994bbe30df22a697b982c8;hpb=946885b04de70f8481f58160de12f3ee3b0b380a;p=tlsproxy%2Ftlsproxy.git

diff --git a/src/verify.c b/src/verify.c
index 0052ca6..14c7127 100644
--- a/src/verify.c
+++ b/src/verify.c
@@ -106,31 +106,11 @@ int verify_tls_connection(gnutls_session_t session, const char *hostname) {
         gnutls_x509_crt_deinit(cert);
         return -1;
     }
-    /* We got the certificate as PEM, free it. */
+    /* We got the certificate as PEM, free the crt struct. */
     gnutls_x509_crt_deinit(cert);
 
-    /* Hostname too long. */
-    if (sizeof(path) - strlen(STORED_SERVER_CERT_FORMAT) <= strlen(hostname)) {
-        LOG(LOG_WARNING,
-            "verify_tls_connection(): hostname too long: '%s'",
-            hostname);
-        return -1;
-    }
-    /* Try to prevent path traversals in hostnames. */
-    if (NULL != strstr(hostname, "..")) {
-        LOG(LOG_WARNING,
-            "verify_tls_connection(): possible path traversal: '%s'",
-            hostname);
-        return -1;
-    }
-    snprintf(path, sizeof(path), STORED_SERVER_CERT_FORMAT, hostname);
-
-    /* Open the stored certificate file. */
-    file = fopen(path, "rb");
-    if (NULL == file) {
-        LOG(LOG_WARNING,
-            "verify_tls_connection(): failed to open '%s': %s",
-            path, strerror(errno));
+    /* Open stored server certificate file. */
+    if (0 != server_certificate_path(&file, hostname, path, sizeof(path))) {
         return -1;
     }
 
@@ -141,8 +121,7 @@ int verify_tls_connection(gnutls_session_t session, const char *hostname) {
         size += strlen(buffer);
         /* Make sure the buffer is big enough. */
         if (sizeof(stored_cert) <= size) {
-            LOG(LOG_WARNING, "verify_tls_connection(): '%s' too big",
-                path);
+            LOG(LOG_WARNING, "verify_tls_connection(): '%s' too big", path);
             fclose(file);
             return -1;
         }
@@ -168,3 +147,34 @@ int verify_tls_connection(gnutls_session_t session, const char *hostname) {
 
     return 0;
 }
+
+int server_certificate_path(FILE **file, const char *hostname,
+                            char *path, size_t size) {
+    /* Hostname too long. */
+    if (size - strlen(STORED_SERVER_CERT_FORMAT) <= strlen(hostname)) {
+        LOG(LOG_WARNING,
+            "server_certificate_path(): hostname too long: '%s'",
+            hostname);
+        return -1;
+    }
+    /* Try to prevent path traversals in hostnames. */
+    if (NULL != strstr(hostname, "..")) {
+        LOG(LOG_WARNING,
+            "server_certificate_path(): possible path traversal: '%s'",
+            hostname);
+        return -1;
+    }
+    snprintf(path, size, STORED_SERVER_CERT_FORMAT, hostname);
+
+    /* Open the stored certificate file. */
+    *file = fopen(path, "rb");
+    if (NULL == *file) {
+        LOG(global_passthrough_unknown ? LOG_DEBUG : LOG_WARNING,
+            "server_certificate_path(): failed to open '%s': %s",
+            path, strerror(errno));
+        /* Couldn't open the file, special case. */
+        return -2;
+    }
+
+    return 0;
+}