Added missing success check around chmod() for windows during temp file creation

diff --git a/main/php_open_temporary_file.c b/main/php_open_temporary_file.c
index a8ea24d1e12c68f79c56b5da93f4563d44ef8b83..458f2bf40563ef5f52be7077c51c01b526cddce4 100644 (file)
--- a/main/php_open_temporary_file.c
+++ b/main/php_open_temporary_file.c
@@ -149,7 +149,11 @@ static int php_do_open_temporary_file(const char *path, const char *pfx, char **
        if (GetTempFileName(new_state.cwd, pfx, 0, opened_path)) {
                /* Some versions of windows set the temp file to be read-only,
                 * which means that opening it will fail... */
-               VCWD_CHMOD(opened_path, 0600);
+               if (VCWD_CHMOD(opened_path, 0600)) {
+                       efree(opened_path);
+                       free(new_state.cwd);
+                       return -1;
+               }
                fd = VCWD_OPEN_MODE(opened_path, open_flags, 0600);
        }