Fix some Typos and indents

Reviewed-by: Andy Polyakov <appro@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/4108)

diff --git a/apps/req.c b/apps/req.c
index f43dae58ab589c9858fc22eaf217205ca915ffc7..dab67a05345475767f06d22a843e8ec83962b5f3 100644 (file)
--- a/apps/req.c
+++ b/apps/req.c
@@ -1305,7 +1305,7 @@ static int check_end(const char *str, const char *end)
 
 /*
  * Merge the two strings together into the result buffer checking for
- * overflow and producing an error message is there is.
+ * overflow and producing an error message if there is.
  */
 static int join(char buf[], size_t buf_size, const char *name,
                 const char *tail, const char *desc)

diff --git a/doc/man3/SSL_CONF_cmd.pod b/doc/man3/SSL_CONF_cmd.pod
index 529acdc356c870f867a7cd1c6e657a60accce169..a8121865a936784eb3938936aea2342027dd03e1 100644 (file)
--- a/doc/man3/SSL_CONF_cmd.pod
+++ b/doc/man3/SSL_CONF_cmd.pod
@@ -480,7 +480,7 @@ SSLv3 is B<always> disabled and attempt to override this by the user are
 ignored.
 
 By checking the return code of SSL_CONF_cmd() it is possible to query if a
-given B<cmd> is recognised, this is useful is SSL_CONF_cmd() values are
+given B<cmd> is recognised, this is useful if SSL_CONF_cmd() values are
 mixed with additional application specific operations.
 
 For example an application might call SSL_CONF_cmd() and if it returns

diff --git a/include/internal/o_dir.h b/include/internal/o_dir.h
index b85275110c7d6c1fea33aa247497815397140a9e..f18fc6708efe3d30356b88311999805a4b77b41f 100644 (file)
--- a/include/internal/o_dir.h
+++ b/include/internal/o_dir.h
@@ -45,12 +45,12 @@ extern "C" {
 
 typedef struct OPENSSL_dir_context_st OPENSSL_DIR_CTX;
 
-  /*
-   * returns NULL on error or end-of-directory. If it is end-of-directory,
-   * errno will be zero
-   */
+/*
+ * returns NULL on error or end-of-directory. If it is end-of-directory,
+ * errno will be zero
+ */
 const char *OPENSSL_DIR_read(OPENSSL_DIR_CTX **ctx, const char *directory);
-  /* returns 1 on success, 0 on error */
+/* returns 1 on success, 0 on error */
 int OPENSSL_DIR_end(OPENSSL_DIR_CTX **ctx);
 
 #ifdef __cplusplus

diff --git a/ssl/ssl_locl.h b/ssl/ssl_locl.h
index c78c28fe45e35219d7b0b3458a76da71ddc20a71..8b8625d7d5a22a9c2ee0154fa37af534f19c8adb 100644 (file)
--- a/ssl/ssl_locl.h
+++ b/ssl/ssl_locl.h
@@ -2364,7 +2364,7 @@ typedef enum ticket_en {
     TICKET_NO_DECRYPT,
     /* a ticket was successfully decrypted */
     TICKET_SUCCESS,
-    /* same as above but the ticket needs to be reneewed */
+    /* same as above but the ticket needs to be renewed */
     TICKET_SUCCESS_RENEW
 } TICKET_RETURN;
 

diff --git a/ssl/statem/extensions.c b/ssl/statem/extensions.c
index f62b1fe65f9e3c56c7e4e4286e8396b103403453..2268b271ce0daf82804a5c1da9e86cc2be52cafc 100644 (file)
--- a/ssl/statem/extensions.c
+++ b/ssl/statem/extensions.c
@@ -720,7 +720,7 @@ int tls_construct_extensions(SSL *s, WPACKET *pkt, unsigned int context,
 
     /* Add custom extensions first */
     if ((context & SSL_EXT_CLIENT_HELLO) != 0) {
-        /* On the server side with initiase during ClientHello parsing */
+        /* On the server side with initialise during ClientHello parsing */
         custom_ext_init(&s->cert->custext);
     }
     if (!custom_ext_add(s, context, pkt, x, chainidx, max_version, &tmpal)) {

diff --git a/ssl/statem/extensions_srvr.c b/ssl/statem/extensions_srvr.c
index a3c2fbf7de516de653de5ccede29edf871081699..a70f53bf3d6c2155eab90fb27e1e98451804b679 100644 (file)
--- a/ssl/statem/extensions_srvr.c
+++ b/ssl/statem/extensions_srvr.c
@@ -87,10 +87,9 @@ int tls_parse_ctos_server_name(SSL *s, PACKET *pkt, unsigned int context,
     }
 
     /*
-     * Although the server_name extension was intended to be
-     * extensible to new name types, RFC 4366 defined the
-     * syntax inextensibly and OpenSSL 1.0.x parses it as
-     * such.
+     * Although the intent was for server_name to be extensible, RFC 4366
+     * was not clear about it; and so OpenSSL among other implementations,
+     * always and only allows a 'host_name' name types.
      * RFC 6066 corrected the mistake but adding new name types
      * is nevertheless no longer feasible, so act as if no other
      * SNI types can exist, to simplify parsing.

diff --git a/test/README.external b/test/README.external
index b4b011b5cab11c31963710a771d5a5c9d0da3289..ab0f7c27910a7144b76f8d930102a4582481298f 100644 (file)
--- a/test/README.external
+++ b/test/README.external
@@ -126,7 +126,7 @@ explicitly run (with more debugging):
 
 $ VERBOSE=1 make TESTS=test_external_krb5 test
 
-Test failures supressions
+Test-failures suppressions
 -------------------------
 
 krb5 will automatically adapt its test suite to account for the configuration

diff --git a/test/handshake_helper.c b/test/handshake_helper.c
index 6584e4db1e99bda6426a51f6784368743fe33843..d200332771f8dd43739f8308679406c34d8c5668 100644 (file)
--- a/test/handshake_helper.c
+++ b/test/handshake_helper.c
@@ -855,7 +855,7 @@ static void do_reneg_setup_step(const SSL_TEST_CTX *test_ctx, PEER *peer)
         if (!SSL_renegotiate_pending(peer->ssl)) {
             /*
              * If we are the client we will always attempt to resume the
-             * session. The server may or may not resume dependant on the
+             * session. The server may or may not resume dependent on the
              * setting of SSL_OP_NO_SESSION_RESUMPTION_ON_RENEGOTIATION
              */
             if (SSL_is_server(peer->ssl)) {

diff --git a/test/secmemtest.c b/test/secmemtest.c
index a6ccc3b26cc409471e6ac4427e174552c150c569..9efa2c89d53d0c060ad74175cb5f911e019029df 100644 (file)
--- a/test/secmemtest.c
+++ b/test/secmemtest.c
@@ -32,7 +32,7 @@ static int test_sec_mem(void)
     if (!TEST_ptr(p)
         /* r = non-secure 20, p = secure 20, s = non-secure 20 */
         || !TEST_true(CRYPTO_secure_allocated(p))
-        /* 20 secure -> 32-byte minimum allocaton unit */
+        /* 20 secure -> 32-byte minimum allocation unit */
         || !TEST_size_t_eq(CRYPTO_secure_used(), 32))
         goto end;
     q = OPENSSL_malloc(20);

diff --git a/test/ssltest_old.c b/test/ssltest_old.c
index 37e980d38f271d1e4d56e974a1a0eeb08367f6eb..3a8d9b26375cd301ccb8fbd95ec09a9a3feca3c2 100644 (file)
--- a/test/ssltest_old.c
+++ b/test/ssltest_old.c
@@ -2144,7 +2144,7 @@ int doit_biopair(SSL *s_ssl, SSL *c_ssl, long count,
          * Useful functions for querying the state of BIO pair endpoints:
          *
          * BIO_ctrl_pending(bio)              number of bytes we can read now
-         * BIO_ctrl_get_read_request(bio)     number of bytes needed to fulfil
+         * BIO_ctrl_get_read_request(bio)     number of bytes needed to fulfill
          *                                      other side's read attempt
          * BIO_ctrl_get_write_guarantee(bio)   number of bytes we can write now
          *

diff --git a/test/tls13secretstest.c b/test/tls13secretstest.c
index 9f32fdf7b289222075e247b6c47070c52b7aca04..1c47e4175c67aa767b9799225c58ce2acb2974bf 100644 (file)
--- a/test/tls13secretstest.c
+++ b/test/tls13secretstest.c
@@ -289,7 +289,7 @@ static int test_handshake_secrets(void)
 
     if (!TEST_true(tls13_generate_handshake_secret(s, ecdhe_secret,
                                                    sizeof(ecdhe_secret)))) {
-        TEST_info("Hanshake secret generation failed");
+        TEST_info("Handshake secret generation failed");
         goto err;
     }
 

diff --git a/util/dofile.pl b/util/dofile.pl
index 8b0c7b41c0667749aa9030252225886ff303210b..0d05574667e463758f91f0da42ca63d58356027b 100644 (file)
--- a/util/dofile.pl
+++ b/util/dofile.pl
@@ -91,7 +91,7 @@ package main;
 # Helper functions for the templates #################################
 
 # It might be practical to quotify some strings and have them protected
-# from possible harm.  These functions primarly quote things that might
+# from possible harm.  These functions primarily quote things that might
 # be interpreted wrongly by a perl eval.
 
 # quotify1 STRING

diff --git a/util/find-doc-nits b/util/find-doc-nits
index 2623d6c958ea753d2af320e3bd852e511eddff81..7f5f1eb06f99ec219830f20b37adffc1d3365c97 100755 (executable)
--- a/util/find-doc-nits
+++ b/util/find-doc-nits
@@ -289,7 +289,7 @@ sub checkmacros()
         }
         close(IN);
     }
-    print "# Found $count macros missing (not all should be documnted)\n"
+    print "# Found $count macros missing (not all should be documented)\n"
 }
 
 sub printem()

diff --git a/util/openssl-format-source b/util/openssl-format-source
index 3dcc128a9f1ec747fac854dd4de0954543fa30df..2655e9c4009ccaa6d5e320c94c7d8c7a21881f72 100755 (executable)
--- a/util/openssl-format-source
+++ b/util/openssl-format-source
@@ -111,7 +111,7 @@ do
            # the process-comments options and then undo that marking, and then 
            # finally re-run indent without process-comments so the marked-to-
            # be-ignored comments we did automatically end up getting moved 
-           # into the right possition within the code as indent leaves marked 
+           # into the right position within the code as indent leaves marked 
            # comments entirely untouched - we appear to have no way to avoid 
            # the double processing and get the desired output
            cat "$j" | \