src/usr.bin/ftp/main.c - diff

Return to main.c CVS log

Up to [local] / src / usr.bin / ftp

Diff for /src/usr.bin/ftp/main.c between version 1.109 and 1.110

-version 1.109, 2016/07/13 16:35:47
+version 1.110, 2016/08/13 12:55:21
 Line 98
 Line 98
 Line 98
  };
  struct tls_config *tls_config;
+ static void
+ process_ssl_options(char *cp)
+ {
+         const char *errstr;
+         long long depth;
+         char *str;
+         while (*cp) {
+                 switch (getsubopt(&cp, ssl_verify_opts, &str)) {
+                 case SSL_CAFILE:
+                         if (str == NULL)
+                                 errx(1, "missing CA file");
+                         if (tls_config_set_ca_file(tls_config, str) != 0)
+                                 errx(1, "tls ca file failed: %s",
+                                     tls_config_error(tls_config));
+                         break;
+                 case SSL_CAPATH:
+                         if (str == NULL)
+                                 errx(1, "missing CA directory path");
+                         if (tls_config_set_ca_path(tls_config, str) != 0)
+                                 errx(1, "tls ca path failed: %s",
+                                     tls_config_error(tls_config));
+                         break;
+                 case SSL_CIPHERS:
+                         if (str == NULL)
+                                 errx(1, "missing cipher list");
+                         if (tls_config_set_ciphers(tls_config, str) != 0)
+                                 errx(1, "tls ciphers failed: %s",
+                                     tls_config_error(tls_config));
+                         break;
+                 case SSL_DONTVERIFY:
+                         tls_config_insecure_noverifycert(tls_config);
+                         tls_config_insecure_noverifyname(tls_config);
+                         break;
+                 case SSL_DOVERIFY:
+                         tls_config_verify(tls_config);
+                         break;
+                 case SSL_VERIFYDEPTH:
+                         if (str == NULL)
+                                 errx(1, "missing depth");
+                         depth = strtonum(str, 0, INT_MAX, &errstr);
+                         if (errstr)
+                                 errx(1, "certificate validation depth is %s",
+                                     errstr);
+                         tls_config_set_verify_depth(tls_config, (int)depth);
+                         break;
+                 default:
+                         errx(1, "unknown -S suboption `%s'",
+                             suboptarg ? suboptarg : "");
+                         /* NOTREACHED */
+                 }
+         }
+ }
  #endif /* !SMALL */
  int family = PF_UNSPEC;
-Line 112
+Line 166
 Line 112
 Line 166
          char *outfile = NULL;
          const char *errstr;
          int dumb_terminal = 0;
- #ifndef SMALL
-         long long depth;
- #endif
          ftpport = "ftp";
          httpport = "http";
-Line 202
+Line 253
 Line 202
 Line 253
                          errx(1, "tls config failed");
                  tls_config_set_protocols(tls_config, TLS_PROTOCOLS_ALL);
                  if (tls_config_set_ciphers(tls_config, "all") != 0)
-                         errx(1, "tls set ciphers failed");
+                         errx(1, "tls set ciphers failed: %s",
+                             tls_config_error(tls_config));
          }
  #endif /* !SMALL */
-Line 318
+Line 370
 Line 318
 Line 370
                  case 'S':
  #ifndef SMALL
-                         cp = optarg;
+                         process_ssl_options(optarg);
-                         while (*cp) {
+ #endif /* !SMALL */
-                                 char    *str;
-                                 switch (getsubopt(&cp, ssl_verify_opts, &str)) {
-                                 case SSL_CAFILE:
-                                         if (str == NULL)
-                                                 errx(1, "missing CA file");
-                                         if (tls_config_set_ca_file(
-                                             tls_config, str) != 0)
-                                                 errx(1, "tls ca file failed");
-                                         break;
-                                 case SSL_CAPATH:
-                                         if (str == NULL)
-                                                 errx(1, "missing CA directory"
-                                                     " path");
-                                         if (tls_config_set_ca_path(
-                                             tls_config, str) != 0)
-                                                 errx(1, "tls ca path failed");
-                                         break;
-                                 case SSL_CIPHERS:
-                                         if (str == NULL)
-                                                 errx(1, "missing cipher list");
-                                         if (tls_config_set_ciphers(
-                                             tls_config, str) != 0)
-                                                 errx(1, "tls ciphers failed");
-                                         break;
-                                 case SSL_DONTVERIFY:
-                                         tls_config_insecure_noverifycert(
-                                             tls_config);
-                                         tls_config_insecure_noverifyname(
-                                             tls_config);
-                                         break;
-                                 case SSL_DOVERIFY:
-                                         tls_config_verify(tls_config);
-                                         break;
-                                 case SSL_VERIFYDEPTH:
-                                         if (str == NULL)
-                                                 errx(1, "missing depth");
-                                         depth = strtonum(str, 0, INT_MAX,
-                                             &errstr);
-                                         if (errstr)
-                                                 errx(1, "certificate "
-                                                     "validation depth is %s",
-                                                     errstr);
-                                         tls_config_set_verify_depth(
-                                             tls_config, (int)depth);
-                                         break;
-                                 default:
-                                         errx(1, "unknown -S suboption `%s'",
-                                             suboptarg ? suboptarg : "");
-                                         /* NOTREACHED */
-                                 }
-                         }
- #endif
                          break;
                  case 's':