]> granicus.if.org Git - php/commitdiff
Avoid compiler warnings in proc_open.c
authorAlex Dowad <alexinbeijing@gmail.com>
Tue, 26 May 2020 13:34:52 +0000 (15:34 +0200)
committerMáté Kocsis <kocsismate@woohoolabs.com>
Wed, 27 May 2020 07:05:14 +0000 (09:05 +0200)
We are getting a lot of compiler warnings when building on Appveyor CI (Windows).
Fix some of them.

Closes GH-5629

ext/standard/proc_open.c

index 94099b7101865b0da44f6518951c6427d29935ff..e4a38851e0a105aa59b2782553e6edf42b8f25d4 100644 (file)
@@ -74,7 +74,7 @@ static php_process_env _php_array_to_envp(zval *environment)
        char **ep;
 #endif
        char *p;
-       size_t cnt, sizeenv = 0;
+       size_t sizeenv = 0;
        HashTable *env_hash; /* temporary PHP array used as helper */
 
        memset(&env, 0, sizeof(env));
@@ -83,7 +83,7 @@ static php_process_env _php_array_to_envp(zval *environment)
                return env;
        }
 
-       cnt = zend_hash_num_elements(Z_ARRVAL_P(environment));
+       uint32_t cnt = zend_hash_num_elements(Z_ARRVAL_P(environment));
 
        if (cnt < 1) {
 #ifndef PHP_WIN32
@@ -794,7 +794,7 @@ static int set_proc_descriptor_from_array(zval *descitem, descriptorspec_item *d
                }
 
                retval = redirect_proc_descriptor(
-                       &descriptors[ndesc], Z_LVAL_P(ztarget), descriptors, ndesc, nindex);
+                       &descriptors[ndesc], (int)Z_LVAL_P(ztarget), descriptors, ndesc, nindex);
        } else if (zend_string_equals_literal(ztype, "null")) {
                /* Set descriptor to blackhole (discard all data written) */
                retval = set_proc_descriptor_to_blackhole(&descriptors[ndesc]);
@@ -829,7 +829,7 @@ static int set_proc_descriptor_from_resource(zval *resource, descriptorspec_item
        }
 
 #ifdef PHP_WIN32
-       php_file_descriptor_t fd_t = (HANDLE)_get_osfhandle(fd);
+       php_file_descriptor_t fd_t = (php_file_descriptor_t)_get_osfhandle(fd);
 #else
        php_file_descriptor_t fd_t = fd;
 #endif
@@ -991,8 +991,8 @@ PHP_FUNCTION(proc_open)
                                goto exit_fail;
                        }
                } else if (Z_TYPE_P(descitem) == IS_ARRAY) {
-                       if (set_proc_descriptor_from_array(descitem, descriptors, ndesc, nindex, &pty_master_fd,
-                               &pty_slave_fd) == FAILURE) {
+                       if (set_proc_descriptor_from_array(descitem, descriptors, ndesc, (int)nindex,
+                               &pty_master_fd, &pty_slave_fd) == FAILURE) {
                                goto exit_fail;
                        }
                } else {