]> granicus.if.org Git - php/commitdiff
Add parenthesis to if statements
authorChad Sikorra <Chad.Sikorra@gmail.com>
Wed, 29 Jun 2016 12:37:48 +0000 (07:37 -0500)
committerChad Sikorra <Chad.Sikorra@gmail.com>
Wed, 29 Jun 2016 12:37:48 +0000 (07:37 -0500)
ext/ldap/ldap.c

index 3b952865c5b735b6bf77011e02b3bd41cf7ee6e9..3c67a5649f3d697eb9ce4e4053d17ca037a9d9af 100644 (file)
@@ -2834,10 +2834,10 @@ static zend_string* php_ldap_do_escape(const zend_bool *map, const char *value,
                len += (map[(unsigned char) value[i]]) ? 3 : 1;
        }
        /* Per RFC 4514, a leading and trailing space must be escaped */
-       if (flags & PHP_LDAP_ESCAPE_DN && value[0] == ' ') {
+       if ((flags & PHP_LDAP_ESCAPE_DN) && (value[0] == ' ')) {
                len += 2;
        }
-       if (flags & PHP_LDAP_ESCAPE_DN && valuelen > 1 && value[valuelen - 1] == ' ') {
+       if ((flags & PHP_LDAP_ESCAPE_DN) && ((valuelen > 1) && (value[valuelen - 1] == ' '))) {
                len += 2;
        }
 
@@ -2846,7 +2846,7 @@ static zend_string* php_ldap_do_escape(const zend_bool *map, const char *value,
        for (i = 0; i < valuelen; i++) {
                unsigned char v = (unsigned char) value[i];
 
-               if (map[v] || (flags & PHP_LDAP_ESCAPE_DN && (i == 0 || i + 1 == valuelen) && v == ' ')) {
+               if (map[v] || ((flags & PHP_LDAP_ESCAPE_DN) && ((i == 0) || (i + 1 == valuelen)) && (v == ' '))) {
                        ZSTR_VAL(ret)[p++] = '\\';
                        ZSTR_VAL(ret)[p++] = hex[v >> 4];
                        ZSTR_VAL(ret)[p++] = hex[v & 0x0f];