]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
MINOR: sample: remove impossible tests on negative smp->data.u.str.data
authorWilly Tarreau <w@1wt.eu>
Wed, 22 Aug 2018 03:07:14 +0000 (05:07 +0200)
committerWilly Tarreau <w@1wt.eu>
Wed, 22 Aug 2018 03:28:33 +0000 (05:28 +0200)
Since commit 843b7cb ("MEDIUM: chunks: make the chunk struct's fields
match the buffer struct") a chunk length is unsigned so we can remove
negative size checks.

include/proto/sample.h

index 6cca99a6a88109d5d7ea6a6238ae17c0cfa00109..4fb5babf4dffd233ee593577706ec94762763582 100644 (file)
@@ -92,8 +92,7 @@ int smp_is_safe(struct sample *smp)
                /* Fall through */
 
        case SMP_T_STR:
-               if ((smp->data.u.str.data < 0) ||
-                   (smp->data.u.str.size && smp->data.u.str.data >= smp->data.u.str.size))
+               if (smp->data.u.str.size && smp->data.u.str.data >= smp->data.u.str.size)
                        return 0;
 
                if (smp->data.u.str.area[smp->data.u.str.data] == 0)
@@ -106,8 +105,7 @@ int smp_is_safe(struct sample *smp)
                return 1;
 
        case SMP_T_BIN:
-               return (smp->data.u.str.data >= 0) &&
-                      (!smp->data.u.str.size || smp->data.u.str.data <= smp->data.u.str.size);
+               return !smp->data.u.str.size || smp->data.u.str.data <= smp->data.u.str.size;
 
        default:
                return 1;
@@ -145,7 +143,6 @@ int smp_is_rw(struct sample *smp)
 
        case SMP_T_STR:
                if (!smp->data.u.str.size ||
-                   smp->data.u.str.data < 0 ||
                    smp->data.u.str.data >= smp->data.u.str.size)
                        return 0;
 
@@ -155,7 +152,6 @@ int smp_is_rw(struct sample *smp)
 
        case SMP_T_BIN:
                return smp->data.u.str.size &&
-                      smp->data.u.str.data >= 0 &&
                       smp->data.u.str.data <= smp->data.u.str.size;
 
        default: