Fix typos.

Typos found with codespell.

Please keep in mind that this should have impact on actual code
and must be carefully evaluated:

src/core/or/lttng_circuit.inc
-    ctf_enum_value("CONTROLER", CIRCUIT_PURPOSE_CONTROLLER)
+    ctf_enum_value("CONTROLLER", CIRCUIT_PURPOSE_CONTROLLER)

28 files changed, 44 insertions, 44 deletions

diff --git a/src/lib/buf/buffers.c b/src/lib/buf/buffers.c
index 23fc1e23a6..e9d5f7f031 100644
--- a/src/lib/buf/buffers.c
+++ b/src/lib/buf/buffers.c
@@ -14,7 +14,7 @@
  *
  * All socket-backed and TLS-based connection_t objects have a pair of
  * buffers: one for incoming data, and one for outcoming data.  These are fed
- * and drained from functions in connection.c, trigged by events that are
+ * and drained from functions in connection.c, triggered by events that are
  * monitored in main.c.
  *
  * This module only handles the buffer implementation itself. To use a buffer
diff --git a/src/lib/conf/conftypes.h b/src/lib/conf/conftypes.h
index 081ebf397f..2207508ef9 100644
--- a/src/lib/conf/conftypes.h
+++ b/src/lib/conf/conftypes.h
@@ -273,7 +273,7 @@ typedef struct config_deprecation_t {
 #endif /* !defined(COCCI) */
 
 /**
- * Validation function: verify whether a configuation object is well-formed
+ * Validation function: verify whether a configuration object is well-formed
  * and consistent.
  *
  * On success, return 0.  On failure, set <b>msg_out</b> to a newly allocated
diff --git a/src/lib/confmgt/typedvar.c b/src/lib/confmgt/typedvar.c
index 1955302cdc..240c03a316 100644
--- a/src/lib/confmgt/typedvar.c
+++ b/src/lib/confmgt/typedvar.c
@@ -156,7 +156,7 @@ typed_var_copy(void *dest, const void *src, const var_type_def_t *def)
   if (BUG(!def))
     return -1; // LCOV_EXCL_LINE
   if (def->fns->copy) {
-    // If we have been provided a copy fuction, use it.
+    // If we have been provided a copy function, use it.
     return def->fns->copy(dest, src, def);
   }
 
diff --git a/src/lib/container/namemap.c b/src/lib/container/namemap.c
index e286cad947..5299c78e83 100644
--- a/src/lib/container/namemap.c
+++ b/src/lib/container/namemap.c
@@ -82,7 +82,7 @@ namemap_fmt_name(const namemap_t *map, unsigned id)
 
 /**
  * Helper: As namemap_get_id(), but requires that <b>name</b> is
- * <b>namelen</b> charaters long, and that <b>namelen</b> is no more than
+ * <b>namelen</b> characters long, and that <b>namelen</b> is no more than
  * MAX_NAMEMAP_NAME_LEN.
  */
 static unsigned
diff --git a/src/lib/crypt_ops/crypto_dh_openssl.c b/src/lib/crypt_ops/crypto_dh_openssl.c
index c5f7271596..f05afd8497 100644
--- a/src/lib/crypt_ops/crypto_dh_openssl.c
+++ b/src/lib/crypt_ops/crypto_dh_openssl.c
@@ -163,7 +163,7 @@ crypto_dh_init_openssl(void)
 
 /** Number of bits to use when choosing the x or y value in a Diffie-Hellman
  * handshake.  Since we exponentiate by this value, choosing a smaller one
- * lets our handhake go faster.
+ * lets our handshake go faster.
  */
 #define DH_PRIVATE_KEY_BITS 320
 
diff --git a/src/lib/crypt_ops/crypto_openssl_mgt.h b/src/lib/crypt_ops/crypto_openssl_mgt.h
index c67ab6467c..91df084a15 100644
--- a/src/lib/crypt_ops/crypto_openssl_mgt.h
+++ b/src/lib/crypt_ops/crypto_openssl_mgt.h
@@ -57,7 +57,7 @@
 #if OPENSSL_VERSION_NUMBER >= OPENSSL_VER(1,1,0,0,5) && \
   !defined(LIBRESSL_VERSION_NUMBER)
 /* OpenSSL as of 1.1.0pre4 has an "new" thread API, which doesn't require
- * seting up various callbacks.
+ * setting up various callbacks.
  *
  * OpenSSL 1.1.0pre4 has a messed up `ERR_remove_thread_state()` prototype,
  * while the previous one was restored in pre5, and the function made a no-op
diff --git a/src/lib/dispatch/dispatch.h b/src/lib/dispatch/dispatch.h
index 9c7c4833c2..63e96bcf8d 100644
--- a/src/lib/dispatch/dispatch.h
+++ b/src/lib/dispatch/dispatch.h
@@ -37,7 +37,7 @@
  * configure messages with their types, channels, and receivers.  Then, use
  * dispatch_new() with that dispatch_cfg_t to create the dispatch_t object.
  *
- * (We use a two-phase contruction procedure here to enable better static
+ * (We use a two-phase construction procedure here to enable better static
  * reasoning about publish/subscribe relationships.)
  *
  * Once you have a dispatch_t, you can queue messages on it with
diff --git a/src/lib/encoding/binascii.c b/src/lib/encoding/binascii.c
index 3e549eb8e3..5f68da183f 100644
--- a/src/lib/encoding/binascii.c
+++ b/src/lib/encoding/binascii.c
@@ -275,7 +275,7 @@ base64_encode(char *dest, size_t destlen, const char *src, size_t srclen,
   }
   switch (n_idx) {
   case 0:
-    /* 0 leftover bits, no pading to add. */
+    /* 0 leftover bits, no padding to add. */
     break;
   case 1:
     /* 8 leftover bits, pad to 12 bits, write the 2 6-bit values followed
diff --git a/src/lib/fs/path.c b/src/lib/fs/path.c
index 2eef4bded7..fc759f6169 100644
--- a/src/lib/fs/path.c
+++ b/src/lib/fs/path.c
@@ -180,7 +180,7 @@ clean_fname_for_stat(char *name)
 /** Modify <b>fname</b> to contain the name of its parent directory.  Doesn't
  * actually examine the filesystem; does a purely syntactic modification.
  *
- * The parent of the root director is considered to be iteself.
+ * The parent of the root director is considered to be itself.
  *
  * Path separators are the forward slash (/) everywhere and additionally
  * the backslash (\) on Win32.
@@ -319,7 +319,7 @@ make_path_absolute(const char *fname)
  * picture explanation here should be read first.
  *
  * Purpose of the functions:
- * - tor_glob - recevies a pattern and returns all the paths that result from
+ * - tor_glob - receives a pattern and returns all the paths that result from
  *   its glob expansion, globs can be present on all path components.
  * - get_glob_opened_files - receives a pattern and returns all the paths that
  *   are opened during its expansion (the paths before any path fragment that
@@ -328,10 +328,10 @@ make_path_absolute(const char *fname)
  *   allowed list.
  *
  * Due to OS API differences explained below, the implementation of tor_glob is
- * completly different for Windows and POSIX systems, so we ended up with three
- * different implementations:
+ * completely different for Windows and POSIX systems, so we ended up with
+ * three different implementations:
  * - tor_glob for POSIX - as POSIX glob does everything we need, we simply call
- *   it and process the results. This is completly implemented in tor_glob.
+ *   it and process the results. This is completely implemented in tor_glob.
  * - tor_glob for WIN32 - because the WIN32 API only supports expanding globs
  *   in the last path fragment, we need to expand the globs in each path
  *   fragment manually and call recursively to get the same behaviour as POSIX
diff --git a/src/lib/log/log.c b/src/lib/log/log.c
index 12e4dbaaa2..411408966b 100644
--- a/src/lib/log/log.c
+++ b/src/lib/log/log.c
@@ -784,7 +784,7 @@ logs_free_all(void)
 
 /** Flush the signal-safe log files.
  *
- * This function is safe to call from a signal handler. It is currenly called
+ * This function is safe to call from a signal handler. It is currently called
  * by the BUG() macros, when terminating the process on an abnormal condition.
  */
 void
diff --git a/src/lib/math/prob_distr.c b/src/lib/math/prob_distr.c
index 31d485120e..b322b4db5c 100644
--- a/src/lib/math/prob_distr.c
+++ b/src/lib/math/prob_distr.c
@@ -891,7 +891,7 @@ icdf_genpareto(double p, double mu, double sigma, double xi)
 
 /**
  * Compute the inverse of the SF of the GeneralizedPareto(mu, sigma,
- * xi) distribution.  Ill-conditioned for p near 1; conditon number is
+ * xi) distribution.  Ill-conditioned for p near 1; condition number is
  *
  *      -xi/(1 - p^{-xi})
  */
@@ -1000,7 +1000,7 @@ sample_uniform_interval(double p0, double a, double b)
      * since if we treat subnormals as having an implicit
      * zero bit before the `binary' point, their exponents
      * are all the same.  There is at most one carry/borrow
-     * bit, which can always be acommodated either in a
+     * bit, which can always be accommodated either in a
      * subnormal, or, at largest, in the implicit one bit
      * of a normal.
      *
diff --git a/src/lib/metrics/metrics_store.c b/src/lib/metrics/metrics_store.c
index abc093a564..57847cc37c 100644
--- a/src/lib/metrics/metrics_store.c
+++ b/src/lib/metrics/metrics_store.c
@@ -26,7 +26,7 @@ struct metrics_store_t {
   /** Indexed by metrics entry name. An entry is a smartlist_t of one or more
    * metrics_store_entry_t allowing for multiple metrics of the same name.
    *
-   * The reason we allow multiple entires is because there are cases where one
+   * The reason we allow multiple entries is because there are cases where one
    * metrics can be used twice by the same entity but with different labels.
    * One example is an onion service with multiple ports, the port specific
    * metrics will have a port value as a label. */
diff --git a/src/lib/net/address.c b/src/lib/net/address.c
index ea6c29db9f..21794fb4fc 100644
--- a/src/lib/net/address.c
+++ b/src/lib/net/address.c
@@ -2126,7 +2126,7 @@ tor_addr_port_copy(tor_addr_port_t *dest,
   memcpy(dest, source, sizeof(tor_addr_port_t));
 }
 
-/** Return true if <b>string</b> represents a valid IPv4 adddress in
+/** Return true if <b>string</b> represents a valid IPv4 address in
  * 'a.b.c.d' form.
  */
 int
diff --git a/src/lib/net/address.h b/src/lib/net/address.h
index bc8ec7744f..4c79db69d2 100644
--- a/src/lib/net/address.h
+++ b/src/lib/net/address.h
@@ -224,7 +224,7 @@ tor_addr_eq_ipv4h(const tor_addr_t *a, uint32_t u)
 #define TOR_ADDR_BUF_LEN 48
 
 /** Length of a buffer containing an IP address along with a port number and
- * a seperating colon.
+ * a separating colon.
  *
  * This allows enough space for
  *   "[ffff:ffff:ffff:ffff:ffff:ffff:255.255.255.255]:12345",
diff --git a/src/lib/process/process.c b/src/lib/process/process.c
index 12c1f9a772..d69b0ca115 100644
--- a/src/lib/process/process.c
+++ b/src/lib/process/process.c
@@ -374,7 +374,7 @@ process_get_protocol(const process_t *process)
   return process->protocol;
 }
 
-/** Set opague pointer to data.  This function allows you to store a pointer to
+/** Set opaque pointer to data.  This function allows you to store a pointer to
  * your own data in the given process. Use <b>process_get_data()</b> in the
  * various callback functions to retrieve the data again.
  *
diff --git a/src/lib/process/process_win32.c b/src/lib/process/process_win32.c
index 8683e17fc2..203df136aa 100644
--- a/src/lib/process/process_win32.c
+++ b/src/lib/process/process_win32.c
@@ -366,7 +366,7 @@ process_win32_write(struct process_t *process, buf_t *buffer)
 
   /* Because of the slightly weird API for WriteFileEx() we must set this to 0
    * before we call WriteFileEx() because WriteFileEx() does not reset the last
-   * error itself when it's succesful. See comment below after the call to
+   * error itself when it's successful. See comment below after the call to
    * GetLastError(). */
   SetLastError(0);
 
@@ -479,9 +479,9 @@ process_win32_trigger_completion_callbacks(void)
   }
 }
 
-/** Start the periodic timer which is reponsible for checking whether processes
- * are still alive and to make sure that the Tor process is periodically being
- * moved into an alertable state. */
+/** Start the periodic timer which is responsible for checking whether
+ * processes are still alive and to make sure that the Tor process is
+ * periodically being moved into an alertable state. */
 void
 process_win32_timer_start(void)
 {
@@ -547,7 +547,7 @@ process_win32_timer_callback(periodic_timer_t *timer, void *data)
       /* If process_win32_timer_test_process() returns true, it means that
        * smartlist_remove() might have been called on the list returned by
        * process_get_all_processes(). We start the loop over again until we
-       * have a succesful run over the entire list where the list was not
+       * have a successful run over the entire list where the list was not
        * modified. */
       if (process_win32_timer_test_process(process)) {
         done = false;
@@ -573,7 +573,7 @@ process_win32_timer_test_process(process_t *process)
   BOOL ret = FALSE;
   DWORD exit_code = 0;
 
-  /* Sometimes the Windows kernel wont give us the EOF/Broken Pipe error
+  /* Sometimes the Windows kernel won't give us the EOF/Broken Pipe error
    * message until some time after the process have actually terminated. We
    * make sure that our ReadFileEx() calls for the process have *all* returned
    * and both standard out and error have been marked as EOF before we try to
@@ -619,7 +619,7 @@ process_win32_timer_test_process(process_t *process)
 
 /** Create a new overlapped named pipe. This function creates a new connected,
  * named, pipe in <b>*read_pipe</b> and <b>*write_pipe</b> if the function is
- * succesful. Returns true on sucess, false on failure. */
+ * successful. Returns true on success, false on failure. */
 STATIC bool
 process_win32_create_pipe(HANDLE *read_pipe,
                           HANDLE *write_pipe,
@@ -840,7 +840,7 @@ process_win32_stdin_write_done(DWORD error_code,
     return;
 
   if (error_code == 0) {
-    /** Our data have been succesfully written. Clear our state and schedule
+    /** Our data have been successfully written. Clear our state and schedule
      * the next write. */
     win32_process->stdin_handle.data_available = 0;
     memset(win32_process->stdin_handle.buffer, 0,
@@ -850,7 +850,7 @@ process_win32_stdin_write_done(DWORD error_code,
     process_notify_event_stdin(process);
   } else if (error_code == ERROR_HANDLE_EOF ||
              error_code == ERROR_BROKEN_PIPE) {
-    /* Our WriteFileEx() call was succesful, but we reached the end of our
+    /* Our WriteFileEx() call was successful, but we reached the end of our
      * file.  We mark our handle as having reached EOF and returns. */
     tor_assert(byte_count == 0);
 
@@ -906,7 +906,7 @@ process_win32_read_from_handle(process_win32_handle_t *handle,
 
   /* Because of the slightly weird API for ReadFileEx() we must set this to 0
    * before we call ReadFileEx() because ReadFileEx() does not reset the last
-   * error itself when it's succesful. See comment below after the call to
+   * error itself when it's successful. See comment below after the call to
    * GetLastError(). */
   SetLastError(0);
 
@@ -972,7 +972,7 @@ process_win32_handle_read_completion(process_win32_handle_t *handle,
   handle->busy = false;
 
   if (error_code == 0) {
-    /* Our ReadFileEx() call was succesful and there is data for us. */
+    /* Our ReadFileEx() call was successful and there is data for us. */
 
     /* This cast should be safe since byte_count should never be larger than
      * BUFFER_SIZE. */
diff --git a/src/lib/process/restrict.c b/src/lib/process/restrict.c
index cd2a1c57b5..bf5ae8e500 100644
--- a/src/lib/process/restrict.c
+++ b/src/lib/process/restrict.c
@@ -44,7 +44,7 @@ tor_disable_debugger_attach(void)
 {
   int r = -1;
   log_debug(LD_CONFIG,
-            "Attemping to disable debugger attachment to Tor for "
+            "Attempting to disable debugger attachment to Tor for "
             "unprivileged users.");
 #if defined(__linux__) && defined(HAVE_SYS_PRCTL_H) \
   && defined(HAVE_PRCTL) && defined(PR_SET_DUMPABLE)
diff --git a/src/lib/process/waitpid.c b/src/lib/process/waitpid.c
index 33798f65f0..014bcf2927 100644
--- a/src/lib/process/waitpid.c
+++ b/src/lib/process/waitpid.c
@@ -113,7 +113,7 @@ clear_waitpid_callback(waitpid_callback_t *ent)
   tor_free(ent);
 }
 
-/** Helper: find the callack for <b>pid</b>; if there is one, run it,
+/** Helper: find the callback for <b>pid</b>; if there is one, run it,
  * reporting the exit status as <b>status</b>. */
 static void
 notify_waitpid_callback_by_pid(pid_t pid, int status)
diff --git a/src/lib/pubsub/pub_binding_st.h b/src/lib/pubsub/pub_binding_st.h
index d7c562fc35..e35f246c57 100644
--- a/src/lib/pubsub/pub_binding_st.h
+++ b/src/lib/pubsub/pub_binding_st.h
@@ -30,7 +30,7 @@ typedef struct pub_binding_t {
   /**
    * A template for the msg_t fields that are filled in for this message.
    * This is copied into outgoing messages, ensuring that their fields are set
-   * corretly.
+   * correctly.
    **/
   msg_t msg_template;
 } pub_binding_t;
diff --git a/src/lib/pubsub/pubsub_build.h b/src/lib/pubsub/pubsub_build.h
index 2781b8251a..87fcaf458e 100644
--- a/src/lib/pubsub/pubsub_build.h
+++ b/src/lib/pubsub/pubsub_build.h
@@ -55,7 +55,7 @@ void pubsub_builder_free_(pubsub_builder_t *);
 
 /**
  * Create a pubsub connector that a single subsystem will use to
- * register its messages.  The main-init code does this during susbsystem
+ * register its messages.  The main-init code does this during subsystem
  * initialization.
  */
 struct pubsub_connector_t *pubsub_connector_for_subsystem(pubsub_builder_t *,
diff --git a/src/lib/pubsub/pubsub_check.c b/src/lib/pubsub/pubsub_check.c
index dbcbb14746..84958e101a 100644
--- a/src/lib/pubsub/pubsub_check.c
+++ b/src/lib/pubsub/pubsub_check.c
@@ -31,7 +31,7 @@ static void pubsub_adjmap_add(pubsub_adjmap_t *map,
                                 const pubsub_cfg_t *item);
 
 /**
- * Helper: contruct and return a new pubsub_adjacency_map from <b>cfg</b>.
+ * Helper: construct and return a new pubsub_adjacency_map from <b>cfg</b>.
  * Return NULL on error.
  **/
 static pubsub_adjmap_t *
diff --git a/src/lib/pubsub/pubsub_macros.h b/src/lib/pubsub/pubsub_macros.h
index e5ffbe501a..4f5b60c19b 100644
--- a/src/lib/pubsub/pubsub_macros.h
+++ b/src/lib/pubsub/pubsub_macros.h
@@ -135,7 +135,7 @@
 #include "lib/pubsub/pubsub_flags.h"
 #include "lib/pubsub/pubsub_publish.h"
 
-/* Implemenation notes:
+/* Implementation notes:
  *
  * For a messagename "foo", the DECLARE_MESSAGE*() macros must declare:
  *
diff --git a/src/lib/subsys/initialization.md b/src/lib/subsys/initialization.md
index 012ab7000d..ee3104324d 100644
--- a/src/lib/subsys/initialization.md
+++ b/src/lib/subsys/initialization.md
@@ -50,7 +50,7 @@ Our current convention is to use the subsystem mechanism to initialize and
 clean up pieces of Tor.  The more recently updated pieces of Tor will use
 this mechanism.  For examples, see e.g. time_sys.c or log_sys.c.
 
-In simplest terms, a **subsytem** is a logically separate part of Tor that
+In simplest terms, a **subsystem** is a logically separate part of Tor that
 can be initialized, shut down, managed, and configured somewhat independently
 of the rest of the program.
 
@@ -59,7 +59,7 @@ initialize it, desconstruct it, and so on. To define a subsystem, we declare
 a `const` instance of subsys_fns_t.  See the documentation for subsys_fns_t
 for a full list of these functions.
 
-After defining a subsytem, it must be inserted in subsystem_list.c.  At that
+After defining a subsystem, it must be inserted in subsystem_list.c.  At that
 point, table-driven mechanisms in subsysmgr.c will invoke its functions when
 appropriate.
 
diff --git a/src/lib/subsys/subsys.h b/src/lib/subsys/subsys.h
index 4cff5f0b6e..abfc82d56a 100644
--- a/src/lib/subsys/subsys.h
+++ b/src/lib/subsys/subsys.h
@@ -194,7 +194,7 @@ typedef struct subsys_fns_t {
 
   /**
    * Return a list of metrics store of this subsystem. This is called
-   * everytime a request arrives on the MetricsPort.
+   * every time a request arrives on the MetricsPort.
    *
    * The list MUST contain metrics_store_t object and contains entries so it
    * can be formatted for the metrics port.
diff --git a/src/lib/time/compat_time.h b/src/lib/time/compat_time.h
index 5089e16ca5..08fd0f0c25 100644
--- a/src/lib/time/compat_time.h
+++ b/src/lib/time/compat_time.h
@@ -60,7 +60,7 @@
  *
  * Q: So, what backends is monotime_coarse using?
  *
- * A: Generally speaking, it uses "whatever monotonic-ish time implemenation
+ * A: Generally speaking, it uses "whatever monotonic-ish time implementation
  * does not require a context switch."  The various implementations provide
  * this by having a view of the current time in a read-only memory page that
  * is updated with a frequency corresponding to the kernel's tick count.
diff --git a/src/lib/time/tvdiff.c b/src/lib/time/tvdiff.c
index cbad5a48b8..14a89bc76e 100644
--- a/src/lib/time/tvdiff.c
+++ b/src/lib/time/tvdiff.c
@@ -30,7 +30,7 @@ tv_secdiff_impl(const struct timeval *start, const struct timeval *end)
   const int64_t s = (int64_t)start->tv_sec;
   const int64_t e = (int64_t)end->tv_sec;
 
-  /* This may not be the most efficient way of implemeting this check,
+  /* This may not be the most efficient way of implementing this check,
    * but it's easy to see that it's correct and doesn't overflow */
 
   if (s > 0 && e < INT64_MIN + s) {
diff --git a/src/lib/trace/debug.h b/src/lib/trace/debug.h
index 84a2867a6d..4ee14fab8d 100644
--- a/src/lib/trace/debug.h
+++ b/src/lib/trace/debug.h
@@ -20,7 +20,7 @@
 /* Send every event to a debug log level. This is useful to debug new trace
  * events without implementing them for a specific event tracing framework.
  *
- * NOTE: arguments can't be used becaue there is no easy generic ways to learn
+ * NOTE: arguments can't be used because there is no easy generic ways to learn
  * their type and amount. It is probably doable with massive C pre-processor
  * trickery but this is meant to be simple. */
 
diff --git a/src/lib/trace/trace_stub.c b/src/lib/trace/trace_stub.c
index 9043efe360..fddf8c63f0 100644
--- a/src/lib/trace/trace_stub.c
+++ b/src/lib/trace/trace_stub.c
@@ -3,7 +3,7 @@
 
 /**
  * \file trace_stub.c
- * \brief Stub declaratinos for use when trace library is disabled.
+ * \brief Stub declarations for use when trace library is disabled.
  **/
 
 #include "lib/subsys/subsys.h"