]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
CLEANUP: Apply ist.cocci
authorTim Duesterhus <tim@bastelstu.be>
Thu, 4 Nov 2021 21:35:44 +0000 (22:35 +0100)
committerWilly Tarreau <w@1wt.eu>
Fri, 5 Nov 2021 06:48:38 +0000 (07:48 +0100)
Make use of the new rules to use `istnext()`.

src/cache.c
src/http_htx.c
src/mqtt.c

index feab63f07f7806d314c13c239ef222addf0c16af..ba2b63c49a05bb67ab3814f3e32d1910d635e18b 100644 (file)
@@ -2180,49 +2180,49 @@ static int parse_encoding_value(struct ist encoding, unsigned int *encoding_valu
 
        switch (*encoding.ptr) {
        case 'a':
-               encoding = istadv(encoding, 1);
+               encoding = istnext(encoding);
                *encoding_value = CHECK_ENCODING(encoding, "aes128gcm", VARY_ENCODING_AES128GCM);
                break;
        case 'b':
-               encoding = istadv(encoding, 1);
+               encoding = istnext(encoding);
                *encoding_value = CHECK_ENCODING(encoding, "br", VARY_ENCODING_BR);
                break;
        case 'c':
-               encoding = istadv(encoding, 1);
+               encoding = istnext(encoding);
                *encoding_value = CHECK_ENCODING(encoding, "compress", VARY_ENCODING_COMPRESS);
                break;
        case 'd':
-               encoding = istadv(encoding, 1);
+               encoding = istnext(encoding);
                *encoding_value = CHECK_ENCODING(encoding, "deflate", VARY_ENCODING_DEFLATE);
                break;
        case 'e':
-               encoding = istadv(encoding, 1);
+               encoding = istnext(encoding);
                *encoding_value = CHECK_ENCODING(encoding, "exi", VARY_ENCODING_EXI);
                break;
        case 'g':
-               encoding = istadv(encoding, 1);
+               encoding = istnext(encoding);
                *encoding_value = CHECK_ENCODING(encoding, "gzip", VARY_ENCODING_GZIP);
                break;
        case 'i':
-               encoding = istadv(encoding, 1);
+               encoding = istnext(encoding);
                *encoding_value = CHECK_ENCODING(encoding, "identity", VARY_ENCODING_IDENTITY);
                break;
        case 'p':
-               encoding = istadv(encoding, 1);
+               encoding = istnext(encoding);
                *encoding_value = CHECK_ENCODING(encoding, "pack200-gzip", VARY_ENCODING_PACK200_GZIP);
                break;
        case 'x':
-               encoding = istadv(encoding, 1);
+               encoding = istnext(encoding);
                *encoding_value = CHECK_ENCODING(encoding, "x-gzip", VARY_ENCODING_GZIP);
                if (!*encoding_value)
                        *encoding_value = CHECK_ENCODING(encoding, "x-compress", VARY_ENCODING_COMPRESS);
                break;
        case 'z':
-               encoding = istadv(encoding, 1);
+               encoding = istnext(encoding);
                *encoding_value = CHECK_ENCODING(encoding, "zstd", VARY_ENCODING_ZSTD);
                break;
        case '*':
-               encoding = istadv(encoding, 1);
+               encoding = istnext(encoding);
                *encoding_value = VARY_ENCODING_STAR;
                break;
        default:
@@ -2238,7 +2238,7 @@ static int parse_encoding_value(struct ist encoding, unsigned int *encoding_valu
                                return -1;
 
                        if (has_null_weight) {
-                               encoding = istadv(encoding, 1);
+                               encoding = istnext(encoding);
 
                                encoding = http_trim_leading_spht(encoding);
 
index bfdcaef8631a29ab8b06d70c3d05730887468984..8028cfc99927bb0bd9df884fe2756f7acd847805 100644 (file)
@@ -146,8 +146,7 @@ static int __http_find_header(const struct htx *htx, const void *pattern, struct
                        goto next_blk;
                /* Skip comma */
                if (*(v.ptr) == ',') {
-                       v.ptr++;
-                       v.len--;
+                       v = istnext(v);
                }
 
                goto return_hdr;
@@ -216,8 +215,7 @@ static int __http_find_header(const struct htx *htx, const void *pattern, struct
                ctx->lws_before = 0;
                ctx->lws_after = 0;
                while (v.len && HTTP_IS_LWS(*v.ptr)) {
-                       v.ptr++;
-                       v.len--;
+                       v = istnext(v);
                        ctx->lws_before++;
                }
                if (!(flags & HTTP_FIND_FL_FULL))
@@ -457,16 +455,14 @@ int http_replace_req_query(struct htx *htx, const struct ist query)
        uri = htx_sl_req_uri(sl);
        q = uri;
        while (q.len > 0 && *(q.ptr) != '?') {
-               q.ptr++;
-               q.len--;
+               q = istnext(q);
        }
 
        /* skip the question mark or indicate that we must insert it
         * (but only if the format string is not empty then).
         */
        if (q.len) {
-               q.ptr++;
-               q.len--;
+               q = istnext(q);
        }
        else if (query.len > 1)
                offset = 0;
index 662420121cf3e8c70056530bca52a87625ca3b4d..ebdb57d4e34e72670df77c9bdb851c676497c350 100644 (file)
@@ -183,7 +183,7 @@ static inline struct ist mqtt_read_1byte_int(struct ist parser, uint8_t *i)
                return IST_NULL;
        if (i)
                *i = (uint8_t)*istptr(parser);
-       parser = istadv(parser, 1);
+       parser = istnext(parser);
        return parser;
 }