]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
CLEANUP: assorted typo fixes in the code and comments
authorIlya Shipitsin <chipitsine@gmail.com>
Fri, 29 Jul 2022 17:26:53 +0000 (22:26 +0500)
committerWilly Tarreau <w@1wt.eu>
Sat, 6 Aug 2022 15:12:51 +0000 (17:12 +0200)
This is 31st iteration of typo fixes

20 files changed:
addons/deviceatlas/dummy/dac.h
doc/configuration.txt
doc/design-thoughts/thread-group.txt
doc/internals/api/appctx.txt
doc/internals/api/pools.txt
include/haproxy/ncbuf-t.h
include/haproxy/proxy-t.h
include/haproxy/task.h
include/haproxy/xprt_quic-t.h
include/haproxy/xprt_quic.h
src/cfgparse.c
src/check.c
src/flt_bwlim.c
src/http_client.c
src/mux_fcgi.c
src/mux_quic.c
src/ncbuf.c
src/quic_sock.c
src/stconn.c
src/xprt_quic.c

index 55bc6c025ba3b237e8ad8417f030553c4e83c5b2..bf166ae547c5eca7116ffc74d8ee797019975995 100644 (file)
@@ -227,7 +227,7 @@ da_status_t da_atlas_open(da_atlas_t *atlas, da_property_decl_t *extra_props, co
  * @brief read from a mapped data which then replace da_atlas_compile call
  *
  * @param dumppath, anonymous if NULL
- * @param map for anonymous, it is the responsability of the caller to unmap it, ignored otherwise
+ * @param map for anonymous, it is the responsibility of the caller to unmap it, ignored otherwise
  * @param maplen for anonymous, it is the size of the mapped data, ignored otherwise
  * @param ptr Pointer dynamically allocated if the mapping happened normally
  * @param len size of the atlas image
index 36046d569c56f56f499fa38ba96834e13b5086e5..95055eb309ffa1ba1d52919788c552c54f1fcee6 100644 (file)
@@ -23091,7 +23091,7 @@ filter bwlim-out <name> limit <size> key <pattern> [table <table>] [min-size <sz
                 be specified for shared bandwidth limitation filters only.
 
     <time>      is the default time period used to evaluate the bandwidth
-                limitation rate. It can be speficied for per-stream bandwidth
+                limitation rate. It can be specified for per-stream bandwidth
                 limitation filters only. It follows the HAProxy time format and
                 is expressed in milliseconds.
 
@@ -23166,7 +23166,7 @@ some experimentations.
         # of 10m/s with all other streams with the same source address.
         filter bwlim-out limit-by-src key src table limit-by-src limit 10m
 
-        # If this filter is enalbed, the stream will be limited to download at 1m/s,
+        # If this filter is enabled, the stream will be limited to download at 1m/s,
         # independently of all other streams.
         filter bwlim-out limit-by-strm default-limit 1m default-period 1s
 
index 8d218e07288fff4ee1ec2fbcc747b0de7391c9e1..9ab1f556d7989c22f78e684031d95d6d4f969d9f 100644 (file)
@@ -319,7 +319,7 @@ thread mask due to takeover ?), the same FD may appear at the same time:
 
   - in the global update_list. In this case, fd_rm_from_fd_list() already
     performs an attachment check, so it's safe to always call it before closing
-    (since noone else may be in the process of changing anything).
+    (since no one else may be in the process of changing anything).
 
 
 ###########################################################
@@ -481,7 +481,7 @@ the FD, and all those with a bit in it ought to do it.
   - thread_mask (in bind_conf->rx_settings) should become an array of
     MAX_TGROUP longs.
   - when parsing "thread 123" or "thread 2/37", the proper bit is set,
-    assuming the array is either a contigous bitfield or a tgroup array.
+    assuming the array is either a contiguous bitfield or a tgroup array.
     An option RX_O_THR_PER_GRP or RX_O_THR_PER_PROC is set depending on
     how the thread num was parsed, so that we reject mixes.
   - end of parsing: entries translated to the cleanest form (to be determined)
index 0597e10d957b7931c7e77df5467f9b0b94d018fa..137ec7b6c3717b75b411dccfb34eb4e95e80f53a 100644 (file)
@@ -20,7 +20,7 @@ context that was allowed to use extra fields of various types. Other applets
 preferred to use their own context definition.
 
 All this resulted in the appctx->ctx to contain a myriad of definitions of
-vairous service contexts, and in some services abusing other services'
+various service contexts, and in some services abusing other services'
 definitions by laziness, and others being extended to use their own definition
 after having run for a long time on the generic types, some of which were not
 noticed and mistakenly used the same storage locations by accident. A massive
@@ -44,7 +44,7 @@ purposed function is provided: "applet_reserve_svcctx()". This function will
 require the caller to indicate how large an area it needs, and will return a
 pointer to this area after checking that it fits. If it does not, haproxy will
 crash. This is purposely done so that it's known during development that if a
-small structure doesn't fit, a differnet approach is required.
+small structure doesn't fit, a different approach is required.
 
 As such, for the vast majority of commands, the process is the following one:
 
index 2ca354c23793ba9d958666fd90fbe1b78590ff31..315d5c4b237e5c6685e7fd7e65773e739d1f1b80 100644 (file)
@@ -573,5 +573,5 @@ CONFIG_HAP_POOL_CLUSTER_SIZE
         This allows one to define the maximum number of objects that will be
         groupped together in an allocation from the shared pool. Values 4 to 8
         have experimentally shown good results with 16 threads. On systems with
-        more cores or losely coupled caches exhibiting slow atomic operations,
+        more cores or loosely coupled caches exhibiting slow atomic operations,
         it could possibly make sense to slightly increase this value.
index 93c5d433e23d3d75caa1b6ac8e642bdffd1b731b..0dd958fbc1f994c5c44efbc00896d65184dfaaf8 100644 (file)
@@ -7,7 +7,7 @@
  * store data in a non-linear way with gaps between them. The buffer is
  * circular and so data may wrapped.
  *
- * The API of <ncbuf> is splitted in two parts. Please refer to the public API
+ * The API of <ncbuf> is split in two parts. Please refer to the public API
  * declared in this header file which should cover all the needs.
  *
  * To minimize the memory footprint, size of data and gaps are inserted in the
index 3d8e0a59e0b51be39fb2a0d5868ad599ccd6ee27..0a94d3ab7b094558841c9d487e4e1f146e5c04d7 100644 (file)
@@ -144,9 +144,9 @@ enum PR_SRV_STATE_FILE {
 
 #define PR_O2_FAKE_KA   0x00200000      /* pretend we do keep-alive with server even though we close */
 
-#define PR_O2_RSTRICT_REQ_HDR_NAMES_BLK  0x00400000 /* reject request with header names containing chars ouside of [0-9a-zA-Z-] charset */
+#define PR_O2_RSTRICT_REQ_HDR_NAMES_BLK  0x00400000 /* reject request with header names containing chars outside of [0-9a-zA-Z-] charset */
 #define PR_O2_RSTRICT_REQ_HDR_NAMES_DEL  0x00800000 /* remove request header names containing chars outside of [0-9a-zA-Z-] charset */
-#define PR_O2_RSTRICT_REQ_HDR_NAMES_NOOP 0x01000000 /* preserve request header names containing chars ouside of [0-9a-zA-Z-] charset */
+#define PR_O2_RSTRICT_REQ_HDR_NAMES_NOOP 0x01000000 /* preserve request header names containing chars outside of [0-9a-zA-Z-] charset */
 #define PR_O2_RSTRICT_REQ_HDR_NAMES_MASK 0x01c00000 /* mask for restrict-http-header-names option */
 /* unused : 0x0000000..0x80000000 */
 
index 2f3fe3d2994d35cc1c2b462d5d07f90709b6a5d5..a9b260aa1c4a2312b7a660b315162f648e6d4837 100644 (file)
@@ -209,7 +209,7 @@ static inline void _task_wakeup(struct task *t, unsigned int f, const char *file
  * happened since the flag was set will result in the task being queued (if
  * it wasn't already). This is used to safely drop the flag from within the
  * scheduler. The flag <f> is combined with existing flags before the test so
- * that it's possible to inconditionally wakeup the task and drop the RUNNING
+ * that it's possible to unconditionally wakeup the task and drop the RUNNING
  * flag if needed.
  */
 #define task_drop_running(t, f) _task_drop_running(t, f, __FILE__, __LINE__)
index f1063ba8a20072219b017d18b7e9ccdfcec4366d..939af7e3a20556563c71800de1453afa47479c23 100644 (file)
@@ -440,9 +440,9 @@ struct quic_rx_crypto_frm {
 #define QUIC_FL_TX_PACKET_PADDING       (1UL << 1)
 /* Flag a sent packet as being in flight. */
 #define QUIC_FL_TX_PACKET_IN_FLIGHT     (QUIC_FL_TX_PACKET_ACK_ELICITING | QUIC_FL_TX_PACKET_PADDING)
-/* Flag a sent packet as containg a CONNECTION_CLOSE frame */
+/* Flag a sent packet as containing a CONNECTION_CLOSE frame */
 #define QUIC_FL_TX_PACKET_CC            (1UL << 2)
-/* Flag a sent packet as containg an ACK frame */
+/* Flag a sent packet as containing an ACK frame */
 #define QUIC_FL_TX_PACKET_ACK           (1UL << 3)
 /* Flag a sent packet as being coalesced to another one in the same datagram */
 #define QUIC_FL_TX_PACKET_COALESCED     (1UL << 4)
index ee08a7d8a6b394427ce5049812d7eccb559ca292..a9e29cbcd055815b59f9a93faab99ca0381e1cc4 100644 (file)
@@ -90,7 +90,7 @@ static inline void quic_cid_cpy(struct quic_cid *dst, const struct quic_cid *src
 }
 
 /* Copy <saddr> socket address data into <buf> buffer.
- * This is the responsability of the caller to check the output buffer is big
+ * This is the responsibility of the caller to check the output buffer is big
  * enough to contain these socket address data.
  * Return the number of bytes copied.
  */
index 1a8926fd183d998c2db3ee5326529b5204e44190..52979390b58c21fa33a369badbf666397a3d1751 100644 (file)
@@ -3790,7 +3790,7 @@ out_uri_auth_compat:
                        if (!bind_conf->mux_proto) {
                                /* No protocol was specified. If we're using QUIC at the transport
                                 * layer, we'll instantiate it as a mux as well. If QUIC is not
-                                * compiled in, this wil remain NULL.
+                                * compiled in, this will remain NULL.
                                 */
                                if (bind_conf->xprt && bind_conf->xprt == xprt_get(XPRT_QUIC))
                                        bind_conf->mux_proto = get_mux_proto(ist("quic"));
index 2366fc5ae0f7921ceeaf5d1d23d9c5e95fc7be96..efca99ce6220447f02c14a75943d000453306e2c 100644 (file)
@@ -1574,7 +1574,7 @@ int init_srv_check(struct server *srv)
        }
        else {
                /* For dynamic servers, check-ssl and check-send-proxy must be
-                * explicitely defined even if the check port was not
+                * explicitly defined even if the check port was not
                 * overridden.
                 */
                if (srv->check.use_ssl == 1)
index c5518b8d4d084d614b234a3ca1f9b3214e7da7e9..e3d3c0dc09ce433a5956a66657a564623fb20055 100644 (file)
@@ -106,7 +106,7 @@ static int bwlim_apply_limit(struct filter *filter, struct channel *chn, unsigne
        else {
                /* On per-stream mode, the freq-counter is private to the
                 * stream. Get it from the filter state. Rely on the custom
-                * limit/period if defined or use the defaut ones. In this mode,
+                * limit/period if defined or use the default ones. In this mode,
                 * there is only one user.
                 */
                bytes_rate = &st->bytes_rate;
@@ -141,7 +141,7 @@ static int bwlim_apply_limit(struct filter *filter, struct channel *chn, unsigne
        if (tokens < len) {
                /* The stream cannot forward all its data. But we will check if
                 * it can perform a small burst if the global quota is large
-                * enought. But, in this case, its waiting time will be
+                * enough. But, in this case, its waiting time will be
                 * increased accordingly.
                 */
                ret = tokens;
@@ -505,7 +505,7 @@ static void release_bwlim_action(struct act_rule *rule)
  *
  *   arg.act.p[0] will be the filter name (mandatory)
  *   arg.act.p[1] will be an expression for the custom limit (optional, may be NULL)
- *   arg.act.p[2] will be an expression for the custom period (optiona, may be NULLl)
+ *   arg.act.p[2] will be an expression for the custom period (optional, may be NULL)
  *
  * It returns ACT_RET_PRS_OK on success, ACT_RET_PRS_ERR on error.
  */
index a1958324313ec6015531bdc22c7d16cf42a22841..cd93ade9a7f87bf350a3ae80c24dd98a4e5a03bf 100644 (file)
@@ -468,7 +468,7 @@ int httpclient_set_dst(struct httpclient *hc, const char *dst)
 }
 
 /*
- * Return a splitted URL in <scheme>, <host>, <port>
+ * Return a split URL in <scheme>, <host>, <port>
  */
 static void httpclient_spliturl(struct ist url, enum http_scheme *scheme,
                                 struct ist *host, int *port)
@@ -877,7 +877,7 @@ static void httpclient_applet_io_handler(struct appctx *appctx)
                                                if (hc->ops.res_payload)
                                                        hc->ops.res_payload(hc);
 
-                                               /* cannot copy everything, need to processs */
+                                               /* cannot copy everything, need to process */
                                                if (vlen != blksz)
                                                        goto process_data;
                                        } else {
index bb9ec5667707177f6793d2b3185fa7e8a3dace1d..6afa9ad8847fb0e79c3ffe0bfbfe1780b998a6d4 100644 (file)
@@ -3858,7 +3858,7 @@ struct task *fcgi_deferred_shut(struct task *t, void *ctx, unsigned int state)
        return NULL;
 }
 
-/* shutr() called by the stream conector (mux_ops.shutr) */
+/* shutr() called by the stream connector (mux_ops.shutr) */
 static void fcgi_shutr(struct stconn *sc, enum co_shr_mode mode)
 {
        struct fcgi_strm *fstrm = __sc_mux_strm(sc);
index e0d074680d914ff620771030fe14b5b0849e6cd1..dd9e9f4a4e7ccfdac446438927025ec06b5e5571 100644 (file)
@@ -451,7 +451,7 @@ static struct ncbuf *qc_get_ncbuf(struct qcs *qcs, struct ncbuf *ncbuf)
        return ncbuf;
 }
 
-/* Notify an eventual subscriber on <qcs> or else wakup up the stconn layer if
+/* Notify an eventual subscriber on <qcs> or else wakeup up the stconn layer if
  * initialized.
  */
 static void qcs_alert(struct qcs *qcs)
@@ -678,7 +678,7 @@ int qcc_get_qcs(struct qcc *qcc, uint64_t id, int receive_only, int send_only,
        }
 
        /* Create the stream. This is valid only for remote initiated one. A
-        * local stream must have already been explicitely created by the
+        * local stream must have already been explicitly created by the
         * application protocol layer.
         */
        if (quic_stream_is_local(qcc, id)) {
@@ -1280,7 +1280,7 @@ static int qcs_build_stream_frm(struct qcs *qcs, struct buffer *out, char fin,
        return -1;
 }
 
-/* Check after transfering data from qcs.tx.buf if FIN must be set on the next
+/* Check after transferring data from qcs.tx.buf if FIN must be set on the next
  * STREAM frame for <qcs>.
  *
  * Returns true if FIN must be set else false.
@@ -1502,7 +1502,7 @@ static int _qc_send_qcs(struct qcs *qcs, struct list *frms)
        /* out buffer cannot be emptied if qcs offsets differ. */
        BUG_ON(!b_data(out) && qcs->tx.sent_offset != qcs->tx.offset);
 
-       /* FIN is set if all incoming data were transfered. */
+       /* FIN is set if all incoming data were transferred. */
        fin = qcs_stream_fin(qcs);
 
        /* Build a new STREAM frame with <out> buffer. */
index d2e17fef475527d975f9dbddcec9e07934675a95..120293944d3a96f59cf8703676c10b5eadf20829 100644 (file)
@@ -506,7 +506,7 @@ int ncb_is_fragmented(const struct ncbuf *buf)
        return !ncb_blk_is_last(buf, gap);
 }
 
-/* Returns the number of bytes of data avaiable in <buf> starting at offset
+/* Returns the number of bytes of data available in <buf> starting at offset
  * <off> until the next gap or the buffer end. The counted data may wrapped if
  * the buffer storage is not aligned.
  */
@@ -589,7 +589,7 @@ enum ncb_ret ncb_add(struct ncbuf *buf, ncb_sz_t off,
                ncb_sz_t done;
 
                /* retrieve the next block. This is necessary to do this
-                * before overwritting a gap.
+                * before overwriting a gap.
                 */
                next = ncb_blk_next(buf, blk);
 
index 3eacb39f22d1d6a038c3445def3ea62a30dba1ee..b7cffc9d23673072c0f0a08eb4193e08d7ab2d40 100644 (file)
@@ -295,7 +295,7 @@ void quic_sock_fd_iocb(int fd)
 
        max_dgrams = global.tune.maxpollevents;
  start:
-       /* Try to reuse an existing dgram. Note that there is alway at
+       /* Try to reuse an existing dgram. Note that there is always at
         * least one datagram to pick, except the first time we enter
         * this function for this <rxbuf> buffer.
         */
@@ -397,7 +397,7 @@ int qc_snd_buf(struct quic_conn *qc, const struct buffer *buf, size_t sz,
                                HA_ATOMIC_INC(&prx_counters->sendto_err);
                }
                else if (errno) {
-                       /* TODO unlisted errno : handle it explicitely. */
+                       /* TODO unlisted errno : handle it explicitly. */
                        ABORT_NOW();
                }
 
index ca36e4958e6bcb772db1617a81604b40149d853e..3ffdf51812981ebacde249567b621c28f7d7fa15 100644 (file)
@@ -244,7 +244,7 @@ static void sc_free_cond(struct stconn **scp)
 
 
 /* Attaches a stconn to a mux endpoint and sets the endpoint ctx. Returns
- * -1 on error and 0 on sucess. SE_FL_DETACHED flag is removed. This function is
+ * -1 on error and 0 on success. SE_FL_DETACHED flag is removed. This function is
  * called from a mux when it is attached to a stream or a health-check.
  */
 int sc_attach_mux(struct stconn *sc, void *sd, void *ctx)
@@ -286,7 +286,7 @@ int sc_attach_mux(struct stconn *sc, void *sd, void *ctx)
 }
 
 /* Attaches a stconn to an applet endpoint and sets the endpoint
- * ctx. Returns -1 on error and 0 on sucess. SE_FL_DETACHED flag is
+ * ctx. Returns -1 on error and 0 on success. SE_FL_DETACHED flag is
  * removed. This function is called by a stream when a backend applet is
  * registered.
  */
@@ -427,7 +427,7 @@ void sc_destroy(struct stconn *sc)
 
 /* Resets the stream connector endpoint. It happens when the app layer want to renew
  * its endpoint. For a connection retry for instance. If a mux or an applet is
- * attached, a new endpoint is created. Returns -1 on error and 0 on sucess.
+ * attached, a new endpoint is created. Returns -1 on error and 0 on success.
  *
  * Only SE_FL_ERROR flag is removed on the endpoint. Orther flags are preserved.
  * It is the caller responsibility to remove other flags if needed.
@@ -1960,7 +1960,7 @@ void sc_conn_prepare_endp_upgrade(struct stconn *sc)
        sc_ep_set(sc, SE_FL_DETACHED);
 }
 
-/* Endpoint upgrade failed. Retore the stconn state. */
+/* Endpoint upgrade failed. Restore the stconn state. */
 void sc_conn_abort_endp_upgrade(struct stconn *sc)
 {
        sc_ep_set(sc, SE_FL_T_MUX);
index 564da9bf078d821e2f3f9cef1edef50917464982..b2431e158f8dbdd7c03dff51eb7f64d648e51ca9 100644 (file)
@@ -1710,7 +1710,7 @@ static inline void qc_requeue_nacked_pkt_tx_frms(struct quic_conn *qc,
 }
 
 /* Free <pkt> TX packet and its attached frames.
- * This is the responsability of the caller to remove this packet of
+ * This is the responsibility of the caller to remove this packet of
  * any data structure it was possibly attached to.
  */
 static inline void free_quic_tx_packet(struct quic_tx_packet *pkt)
@@ -2642,7 +2642,7 @@ static int qc_parse_pkt_frms(struct quic_rx_packet *pkt, struct ssl_sock_ctx *ct
 /* Must be called only by a <cbuf> writer (packet builder).
  * Return 1 if <cbuf> may be reused to build packets, depending on its <rd> and
  * <wr> internal indexes, 0 if not. When this is the case, reset <wr> writer
- * index after having marked the end of written data. This the responsability
+ * index after having marked the end of written data. This the responsibility
  * of the caller to ensure there is enough room in <cbuf> to write the end of
  * data made of a uint16_t null field.
  *
@@ -4832,7 +4832,7 @@ static int send_stateless_reset(struct listener *l, struct sockaddr_storage *dst
 
 /* QUIC server only function.
  * Add AAD to <add> buffer from <cid> connection ID and <addr> socket address.
- * This is the responsability of the caller to check <aad> size is big enough
+ * This is the responsibility of the caller to check <aad> size is big enough
  * to contain these data.
  * Return the number of bytes copied to <aad>.
  */
@@ -5253,7 +5253,7 @@ static int qc_conn_alloc_ssl_ctx(struct quic_conn *qc)
 }
 
 /* Check that all the bytes between <buf> included and <end> address
- * excluded are null. This is the responsability of the caller to
+ * excluded are null. This is the responsibility of the caller to
  * check that there is at least one byte between <buf> end <end>.
  * Return 1 if this all the bytes are null, 0 if not.
  */
@@ -5271,7 +5271,7 @@ static inline int quic_padding_check(const unsigned char *buf,
  * with the information collected from the packet.
  * This function sets ->len <pkt> field value to the end of the packet past one
  * byte to enable the caller to run this function again to continue to parse
- * the remaing QUIC packets carried by the datagram.
+ * the remaining QUIC packets carried by the datagram.
  * Note that this function always sets this ->len value. If a paquet could
  * not be correctly found, ->len value will be set to the remaining number
  * bytes in the datagram to entirely consume this latter.
@@ -6623,7 +6623,7 @@ struct task *quic_lstnr_dghdlr(struct task *t, void *ctx, unsigned int state)
        return t;
 }
 
-/* Retreive the DCID from a QUIC datagram or packet with <buf> as first octet.
+/* Retrieve the DCID from a QUIC datagram or packet with <buf> as first octet.
  * Returns 1 if succeeded, 0 if not.
  */
 int quic_get_dgram_dcid(unsigned char *buf, const unsigned char *end,