Zend: Refactor virtual_unlink() to take path lengths

Girgias · Girgias · commit 1bca72862840 · 2024-08-23T16:02:42.000+02:00
diff --git a/Zend/zend_virtual_cwd.c b/Zend/zend_virtual_cwd.c
@@ -1575,22 +1575,21 @@ CWD_API int virtual_lstat(const char *path, zend_stat_t *buf) /* {{{ */
 }
 /* }}} */
 
-CWD_API int virtual_unlink(const char *path) /* {{{ */
+CWD_API zend_result virtual_unlink(const char *path, size_t path_length) /* {{{ */
 {
 	cwd_state new_state;
-	int retval;
-	size_t path_length = strlen(path);
+	zend_result retval;
 
 	CWD_STATE_COPY(&new_state, &CWDG(cwd));
 	if (virtual_file_ex(&new_state, path, path_length, NULL, CWD_EXPAND)) {
 		CWD_STATE_FREE_ERR(&new_state);
-		return -1;
+		return FAILURE;
 	}
 
 #ifdef ZEND_WIN32
-	retval = php_win32_ioutil_unlink(new_state.cwd);
+	retval = php_win32_ioutil_unlink(new_state.cwd, new_state.cwd_length);
 #else
-	retval = unlink(new_state.cwd);
+	retval = virtual_unlink_native(new_state.cwd, new_state.cwd_length);
 #endif
 
 	CWD_STATE_FREE_ERR(&new_state);
diff --git a/Zend/zend_virtual_cwd.h b/Zend/zend_virtual_cwd.h
@@ -183,7 +183,7 @@ CWD_API int virtual_creat(const char *path, mode_t mode);
 CWD_API zend_result virtual_rename(const char *old_name, size_t old_name_len, const char *new_name, size_t new_name_len);
 CWD_API int virtual_stat(const char *path, zend_stat_t *buf);
 CWD_API int virtual_lstat(const char *path, zend_stat_t *buf);
-CWD_API int virtual_unlink(const char *path);
+CWD_API zend_result virtual_unlink(const char *path, size_t path_len);
 CWD_API int virtual_mkdir(const char *pathname, mode_t mode);
 CWD_API zend_result virtual_rmdir(const char *path, size_t path_len);
 CWD_API DIR *virtual_opendir(const char *pathname);
@@ -260,6 +260,10 @@ static zend_always_inline zend_result virtual_rename_native(const char *old_name
 	return (rename(old_name, new_name) == 0) ? SUCCESS : FAILURE;
 }
 
+static zend_always_inline zend_result virtual_unlink_native(const char *path, ZEND_ATTRIBUTE_UNUSED size_t path_len) {
+	return (unlink(path) == 0) ? SUCCESS : FAILURE;
+}
+
 static zend_always_inline zend_result virtual_rmdir_native(const char *path, ZEND_ATTRIBUTE_UNUSED size_t path_len) {
 	return (rmdir(path) == 0) ? SUCCESS : FAILURE;
 }
@@ -288,7 +292,7 @@ extern void virtual_cwd_main_cwd_init(uint8_t);
 #define VCWD_RENAME(old_name, old_name_length, new_name, new_name_length) virtual_rename(old_name, old_name_length, new_name, new_name_length)
 #define VCWD_STAT(path, buff) virtual_stat(path, buff)
 # define VCWD_LSTAT(path, buff) virtual_lstat(path, buff)
-#define VCWD_UNLINK(path) virtual_unlink(path)
+#define VCWD_UNLINK(path, path_len) virtual_unlink(path, path_len)
 #define VCWD_MKDIR(path, mode) virtual_mkdir(path, mode)
 #define VCWD_RMDIR(path, path_len) virtual_rmdir(path, path_len)
 #define VCWD_OPENDIR(pathname) virtual_opendir(pathname)
@@ -317,7 +321,7 @@ extern void virtual_cwd_main_cwd_init(uint8_t);
 #define VCWD_RENAME(old_name, old_name_length, new_name, new_name_length) php_win32_ioutil_rename(old_name, old_name_length, new_name, new_name_length)
 #define VCWD_MKDIR(path, mode) php_win32_ioutil_mkdir(path, mode)
 #define VCWD_RMDIR(path, path_length) php_win32_ioutil_rmdir(path, path_length)
-#define VCWD_UNLINK(path) php_win32_ioutil_unlink(path)
+#define VCWD_UNLINK(path, path_length) php_win32_ioutil_unlink(path, path_length)
 #define VCWD_CHDIR(path) php_win32_ioutil_chdir(path)
 #define VCWD_ACCESS(pathname, mode) tsrm_win32_access(pathname, mode)
 #define VCWD_GETCWD(buff, size) php_win32_ioutil_getcwd(buff, size)
@@ -329,7 +333,7 @@ extern void virtual_cwd_main_cwd_init(uint8_t);
 #define VCWD_RENAME(old_name, old_name_len, new_name, new_name_len) virtual_rename_native(old_name, old_name_len, new_name, new_name_len)
 #define VCWD_MKDIR(path, mode) mkdir(path, mode)
 #define VCWD_RMDIR(path, path_len) virtual_rmdir_native(path, path_len)
-#define VCWD_UNLINK(path) unlink(path)
+#define VCWD_UNLINK(path, path_len) virtual_unlink_native(path, path_len)
 #define VCWD_CHDIR(path) chdir(path)
 #define VCWD_ACCESS(pathname, mode) access(pathname, mode)
 #define VCWD_GETCWD(buff, size) getcwd(buff, size)
diff --git a/ext/bz2/bz2.c b/ext/bz2/bz2.c
@@ -231,7 +231,7 @@ PHP_BZ2_API php_stream *_php_stream_bz2open(php_stream_wrapper *wrapper,
 		 * failed.
 		 */
 		if (opened_path && !bz_file && mode[0] == 'w') {
-			VCWD_UNLINK(ZSTR_VAL(*opened_path));
+			VCWD_UNLINK(ZSTR_VAL(*opened_path), ZSTR_LEN(*opened_path));
 		}
 	}
 
diff --git a/ext/ftp/php_ftp.c b/ext/ftp/php_ftp.c
@@ -682,11 +682,12 @@ PHP_FUNCTION(ftp_get)
 	ftpbuf_t	*ftp;
 	ftptype_t	xtype;
 	php_stream	*outstream;
-	char		*local, *remote;
-	size_t		local_len, remote_len;
+	zend_string *local;
+	char *remote;
+	size_t remote_len;
 	zend_long		mode=FTPTYPE_IMAGE, resumepos=0;
 
-	if (zend_parse_parameters(ZEND_NUM_ARGS(), "Opp|ll", &z_ftp, php_ftp_ce, &local, &local_len, &remote, &remote_len, &mode, &resumepos) == FAILURE) {
+	if (zend_parse_parameters(ZEND_NUM_ARGS(), "OPp|ll", &z_ftp, php_ftp_ce, &local, &remote, &remote_len, &mode, &resumepos) == FAILURE) {
 		RETURN_THROWS();
 	}
 	GET_FTPBUF(ftp, z_ftp);
@@ -702,9 +703,9 @@ PHP_FUNCTION(ftp_get)
 #endif
 
 	if (ftp->autoseek && resumepos) {
-		outstream = php_stream_open_wrapper(local, mode == FTPTYPE_ASCII ? "rt+" : "rb+", REPORT_ERRORS, NULL);
+		outstream = php_stream_open_wrapper(ZSTR_VAL(local), mode == FTPTYPE_ASCII ? "rt+" : "rb+", REPORT_ERRORS, NULL);
 		if (outstream == NULL) {
-			outstream = php_stream_open_wrapper(local, mode == FTPTYPE_ASCII ? "wt" : "wb", REPORT_ERRORS, NULL);
+			outstream = php_stream_open_wrapper(ZSTR_VAL(local), mode == FTPTYPE_ASCII ? "wt" : "wb", REPORT_ERRORS, NULL);
 		}
 		if (outstream != NULL) {
 			/* if autoresume is wanted seek to end */
@@ -716,17 +717,17 @@ PHP_FUNCTION(ftp_get)
 			}
 		}
 	} else {
-		outstream = php_stream_open_wrapper(local, mode == FTPTYPE_ASCII ? "wt" : "wb", REPORT_ERRORS, NULL);
+		outstream = php_stream_open_wrapper(ZSTR_VAL(local), mode == FTPTYPE_ASCII ? "wt" : "wb", REPORT_ERRORS, NULL);
 	}
 
 	if (outstream == NULL)	{
-		php_error_docref(NULL, E_WARNING, "Error opening %s", local);
+		php_error_docref(NULL, E_WARNING, "Error opening %s", ZSTR_VAL(local));
 		RETURN_FALSE;
 	}
 
 	if (!ftp_get(ftp, outstream, remote, remote_len, xtype, resumepos)) {
 		php_stream_close(outstream);
-		VCWD_UNLINK(local);
+		VCWD_UNLINK(ZSTR_VAL(local), ZSTR_LEN(local));
 		if (*ftp->inbuf) {
 			php_error_docref(NULL, E_WARNING, "%s", ftp->inbuf);
 		}
@@ -745,12 +746,13 @@ PHP_FUNCTION(ftp_nb_get)
 	ftpbuf_t	*ftp;
 	ftptype_t	xtype;
 	php_stream	*outstream;
-	char		*local, *remote;
-	size_t		local_len, remote_len;
+	zend_string *local;
+	char *remote;
+	size_t remote_len;
 	int ret;
 	zend_long		mode=FTPTYPE_IMAGE, resumepos=0;
 
-	if (zend_parse_parameters(ZEND_NUM_ARGS(), "Oss|ll", &z_ftp, php_ftp_ce, &local, &local_len, &remote, &remote_len, &mode, &resumepos) == FAILURE) {
+	if (zend_parse_parameters(ZEND_NUM_ARGS(), "OSs|ll", &z_ftp, php_ftp_ce, &local, &remote, &remote_len, &mode, &resumepos) == FAILURE) {
 		RETURN_THROWS();
 	}
 	GET_FTPBUF(ftp, z_ftp);
@@ -764,9 +766,9 @@ PHP_FUNCTION(ftp_nb_get)
 	mode = FTPTYPE_IMAGE;
 #endif
 	if (ftp->autoseek && resumepos) {
-		outstream = php_stream_open_wrapper(local, mode == FTPTYPE_ASCII ? "rt+" : "rb+", REPORT_ERRORS, NULL);
+		outstream = php_stream_open_wrapper(ZSTR_VAL(local), mode == FTPTYPE_ASCII ? "rt+" : "rb+", REPORT_ERRORS, NULL);
 		if (outstream == NULL) {
-			outstream = php_stream_open_wrapper(local, mode == FTPTYPE_ASCII ? "wt" : "wb", REPORT_ERRORS, NULL);
+			outstream = php_stream_open_wrapper(ZSTR_VAL(local), mode == FTPTYPE_ASCII ? "wt" : "wb", REPORT_ERRORS, NULL);
 		}
 		if (outstream != NULL) {
 			/* if autoresume is wanted seek to end */
@@ -778,7 +780,7 @@ PHP_FUNCTION(ftp_nb_get)
 			}
 		}
 	} else {
-		outstream = php_stream_open_wrapper(local, mode == FTPTYPE_ASCII ? "wt" : "wb", REPORT_ERRORS, NULL);
+		outstream = php_stream_open_wrapper(ZSTR_VAL(local), mode == FTPTYPE_ASCII ? "wt" : "wb", REPORT_ERRORS, NULL);
 	}
 
 	if (outstream == NULL)	{
@@ -793,7 +795,7 @@ PHP_FUNCTION(ftp_nb_get)
 	if ((ret = ftp_nb_get(ftp, outstream, remote, remote_len, xtype, resumepos)) == PHP_FTP_FAILED) {
 		php_stream_close(outstream);
 		ftp->stream = NULL;
-		VCWD_UNLINK(local);
+		VCWD_UNLINK(ZSTR_VAL(local), ZSTR_LEN(local));
 		if (*ftp->inbuf) {
 			php_error_docref(NULL, E_WARNING, "%s", ftp->inbuf);
 		}
diff --git a/ext/gd/gd.c b/ext/gd/gd.c
@@ -1847,7 +1847,7 @@ static void _php_image_output(INTERNAL_FUNCTION_PARAMETERS, int image_type, cons
 		}
 
 		fclose(tmp);
-		VCWD_UNLINK((const char *)ZSTR_VAL(path)); /* make sure that the temporary file is removed */
+		VCWD_UNLINK(ZSTR_VAL(path), ZSTR_LEN(path)); /* make sure that the temporary file is removed */
 		zend_string_release_ex(path, 0);
 	}
 	RETURN_TRUE;
diff --git a/ext/session/mod_files.c b/ext/session/mod_files.c
@@ -315,12 +315,13 @@ static int ps_files_cleanup_dir(const zend_string *dirname, zend_long maxlifetim
 				memcpy(buf + ZSTR_LEN(dirname) + 1, entry->d_name, entry_len);
 
 				/* NUL terminate it and */
-				buf[ZSTR_LEN(dirname) + entry_len + 1] = '\0';
+				size_t buf_len = ZSTR_LEN(dirname) + entry_len + 1;
+				buf[buf_len] = '\0';
 
 				/* check whether its last access was more than maxlifetime ago */
 				if (VCWD_STAT(buf, &sbuf) == 0 &&
 						(now - sbuf.st_mtime) > maxlifetime) {
-					VCWD_UNLINK(buf);
+					VCWD_UNLINK(buf, buf_len);
 					nrdels++;
 				}
 			}
@@ -602,7 +603,8 @@ PS_DESTROY_FUNC(files)
 	if (data->fd != -1) {
 		ps_files_close(data);
 
-		if (VCWD_UNLINK(buf) == -1) {
+		size_t buf_len = strlen(buf);
+		if (VCWD_UNLINK(buf, buf_len) == FAILURE) {
 			/* This is a little safety check for instances when we are dealing with a regenerated session
 			 * that was not yet written to disk. */
 			if (!VCWD_ACCESS(buf, F_OK)) {
diff --git a/ext/soap/php_sdl.c b/ext/soap/php_sdl.c
@@ -2364,7 +2364,7 @@ static void add_sdl_to_cache(const char *fn, size_t fn_len, const char *uri, tim
 	if (valid_file) {
 		/* This is allowed to fail, this means that another process was raced to create the file. */
 		if (VCWD_RENAME(ZSTR_VAL(temp_file_path), ZSTR_LEN(temp_file_path), fn, fn_len) < 0) {
-			VCWD_UNLINK(ZSTR_VAL(temp_file_path));
+			VCWD_UNLINK(ZSTR_VAL(temp_file_path), ZSTR_LEN(temp_file_path));
 		}
 	}
 
diff --git a/ext/standard/basic_functions.c b/ext/standard/basic_functions.c
@@ -2393,7 +2393,7 @@ PHP_FUNCTION(move_uploaded_file)
 		}
 #endif
 	} else if (php_copy_file_ex(ZSTR_VAL(path), new_path, STREAM_DISABLE_OPEN_BASEDIR) == SUCCESS) {
-		VCWD_UNLINK(ZSTR_VAL(path));
+		VCWD_UNLINK(ZSTR_VAL(path), ZSTR_LEN(path));
 		successful = 1;
 	}
 
diff --git a/main/rfc1867.c b/main/rfc1867.c
@@ -171,7 +171,7 @@ PHPAPI void destroy_uploaded_files_hash(void) /* {{{ */
 
 	ZEND_HASH_MAP_FOREACH_VAL(SG(rfc1867_uploaded_files), el) {
 		zend_string *filename = Z_STR_P(el);
-		VCWD_UNLINK(ZSTR_VAL(filename));
+		VCWD_UNLINK(ZSTR_VAL(filename), ZSTR_LEN(filename));
 	} ZEND_HASH_FOREACH_END();
 	zend_hash_destroy(SG(rfc1867_uploaded_files));
 	FREE_HASHTABLE(SG(rfc1867_uploaded_files));
@@ -1089,6 +1089,7 @@ SAPI_API SAPI_POST_HANDLER_FUNC(rfc1867_post_handler)
 			if (cancel_upload) {
 				if (temp_filename) {
 					if (cancel_upload != PHP_UPLOAD_ERROR_E) { /* file creation failed */
+						// TODO Should this use VCWD_UNLINK()?
 						unlink(ZSTR_VAL(temp_filename));
 					}
 					zend_string_release_ex(temp_filename, 0);
diff --git a/main/streams/plain_wrapper.c b/main/streams/plain_wrapper.c
@@ -509,7 +509,7 @@ static int php_stdiop_close(php_stream *stream, int close_handle)
 		}
 		if (data->temp_name) {
 #ifdef PHP_WIN32
-			php_win32_ioutil_unlink(ZSTR_VAL(data->temp_name));
+			php_win32_ioutil_unlink(ZSTR_VAL(data->temp_name), ZSTR_LEN(data->temp_name));
 #else
 			unlink(ZSTR_VAL(data->temp_name));
 #endif
@@ -1242,8 +1242,6 @@ static int php_plain_files_url_stater(php_stream_wrapper *wrapper, const char *u
 
 static int php_plain_files_unlink(php_stream_wrapper *wrapper, const char *url, int options, php_stream_context *context)
 {
-	int ret;
-
 	if (strncasecmp(url, "file://", sizeof("file://") - 1) == 0) {
 		url += sizeof("file://") - 1;
 	}
@@ -1252,8 +1250,9 @@ static int php_plain_files_unlink(php_stream_wrapper *wrapper, const char *url,
 		return 0;
 	}
 
-	ret = VCWD_UNLINK(url);
-	if (ret == -1) {
+	size_t url_len = strlen(url);
+	zend_result ret = VCWD_UNLINK(url, url_len);
+	if (ret == FAILURE) {
 		if (options & REPORT_ERRORS) {
 			php_error_docref1(NULL, url, E_WARNING, "%s", strerror(errno));
 		}
@@ -1339,7 +1338,7 @@ static bool php_plain_files_rename(php_stream_wrapper *wrapper, const zend_strin
 					}
 #  endif
 					if (success) {
-						VCWD_UNLINK(url_from_ptr);
+						VCWD_UNLINK(url_from_ptr, url_from_len);
 					}
 				} else {
 					php_error_docref2(NULL, ZSTR_VAL(url_from), ZSTR_VAL(url_to), E_WARNING, "%s", strerror(errno));
diff --git a/win32/ioutil.c b/win32/ioutil.c
@@ -365,15 +365,15 @@ PW32IO int php_win32_ioutil_mkdir_w(const wchar_t *path, mode_t mode)
 	return 0;
 }/*}}}*/
 
-PW32IO int php_win32_ioutil_unlink_w(const wchar_t *path)
+PW32IO zend_result php_win32_ioutil_unlink_w(const wchar_t *path)
 {/*{{{*/
 	DWORD err = 0;
 	HANDLE h;
 	BY_HANDLE_FILE_INFORMATION info;
 	FILE_DISPOSITION_INFO disposition;
 	BOOL status;
 
-	PHP_WIN32_IOUTIL_CHECK_PATH_W(path, -1, 0)
+	PHP_WIN32_IOUTIL_CHECK_PATH_W(path, FAILURE, 0)
 
 	h = CreateFileW(path,
 					FILE_READ_ATTRIBUTES | FILE_WRITE_ATTRIBUTES | DELETE,
@@ -386,21 +386,21 @@ PW32IO int php_win32_ioutil_unlink_w(const wchar_t *path)
 	if (INVALID_HANDLE_VALUE == h) {
 		err = GetLastError();
 		SET_ERRNO_FROM_WIN32_CODE(err);
-		return -1;
+		return FAILURE;
 	}
 
 	if (!GetFileInformationByHandle(h, &info)) {
 		err = GetLastError();
 		CloseHandle(h);
 		SET_ERRNO_FROM_WIN32_CODE(err);
-		return -1;
+		return FAILURE;
 	}
 
 	if (info.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY) {
 		/* TODO Handle possible reparse point. */
 		CloseHandle(h);
 		SET_ERRNO_FROM_WIN32_CODE(ERROR_DIRECTORY_NOT_SUPPORTED);
-		return -1;
+		return FAILURE;
 	}
 
 #if 0
@@ -428,12 +428,12 @@ PW32IO int php_win32_ioutil_unlink_w(const wchar_t *path)
 		err = GetLastError();
 		CloseHandle(h);
 		SET_ERRNO_FROM_WIN32_CODE(err);
-		return -1;
+		return FAILURE;
 	}
 
 	CloseHandle(h);
 
-	return 0;
+	return SUCCESS;
 }/*}}}*/
 
 PW32IO int php_win32_ioutil_chdir_w(const wchar_t *path)
diff --git a/win32/ioutil.h b/win32/ioutil.h
@@ -266,7 +266,7 @@ PW32IO int php_win32_ioutil_open_w(const wchar_t *path, int flags, ...);
 PW32IO int php_win32_ioutil_chdir_w(const wchar_t *path);
 PW32IO zend_result php_win32_ioutil_rename_w(const wchar_t *oldname, const wchar_t *newname);
 PW32IO wchar_t *php_win32_ioutil_getcwd_w(wchar_t *buf, size_t len);
-PW32IO int php_win32_ioutil_unlink_w(const wchar_t *path);
+PW32IO zend_result php_win32_ioutil_unlink_w(const wchar_t *path);
 PW32IO int php_win32_ioutil_access_w(const wchar_t *path, mode_t mode);
 PW32IO int php_win32_ioutil_mkdir_w(const wchar_t *path, mode_t mode);
 PW32IO FILE *php_win32_ioutil_fopen_w(const wchar_t *path, const wchar_t *mode);
@@ -335,26 +335,21 @@ __forceinline static int php_win32_ioutil_open(const char *path, int flags, ...)
 	return ret;
 }/*}}}*/
 
-__forceinline static int php_win32_ioutil_unlink(const char *path)
+__forceinline static zend_result php_win32_ioutil_unlink(const char *path, size_t path_len)
 {/*{{{*/
 	PHP_WIN32_IOUTIL_INIT_W(path)
-	int ret = -1;
-	DWORD err;
 
 	if (!pathw) {
 		SET_ERRNO_FROM_WIN32_CODE(ERROR_INVALID_PARAMETER);
-		return -1;
-	}
-
-	ret = php_win32_ioutil_unlink_w(pathw);
-	if (0 > ret) {
-		err = GetLastError();
+		return FAILURE;
 	}
-	PHP_WIN32_IOUTIL_CLEANUP_W()
 
-	if (0 > ret) {
+	zend_result ret = php_win32_ioutil_unlink_w(pathw);
+	if (ret == FAILURE) {
+		DWORD err = GetLastError();
 		SET_ERRNO_FROM_WIN32_CODE(err);
 	}
+	PHP_WIN32_IOUTIL_CLEANUP_W()
 
 	return ret;
 }/*}}}*/

Original file line number	Diff line number	Diff line change
`@@ -231,7 +231,7 @@ PHP_BZ2_API php_stream _php_stream_bz2open(php_stream_wrapper wrapper,`
`231`	`231`	`* failed.`
`232`	`232`	`*/`
`233`	`233`	`if (opened_path && !bz_file && mode[0] == 'w') {`
`234`		`- VCWD_UNLINK(ZSTR_VAL(*opened_path));`
	`234`	`+ VCWD_UNLINK(ZSTR_VAL(opened_path), ZSTR_LEN(opened_path));`
`235`	`235`	`}`
`236`	`236`	`}`
`237`	`237`
Original file line number	Diff line number	Diff line change
`@@ -1847,7 +1847,7 @@ static void _php_image_output(INTERNAL_FUNCTION_PARAMETERS, int image_type, cons`
`1847`	`1847`	`}`
`1848`	`1848`
`1849`	`1849`	`fclose(tmp);`
`1850`		`- VCWD_UNLINK((const char )ZSTR_VAL(path)); / make sure that the temporary file is removed */`
	`1850`	`+ VCWD_UNLINK(ZSTR_VAL(path), ZSTR_LEN(path)); /* make sure that the temporary file is removed */`
`1851`	`1851`	`zend_string_release_ex(path, 0);`
`1852`	`1852`	`}`
`1853`	`1853`	`RETURN_TRUE;`
Original file line number	Diff line number	Diff line change
`@@ -2364,7 +2364,7 @@ static void add_sdl_to_cache(const char fn, size_t fn_len, const char uri, tim`
`2364`	`2364`	`if (valid_file) {`
`2365`	`2365`	`/* This is allowed to fail, this means that another process was raced to create the file. */`
`2366`	`2366`	`if (VCWD_RENAME(ZSTR_VAL(temp_file_path), ZSTR_LEN(temp_file_path), fn, fn_len) < 0) {`
`2367`		`- VCWD_UNLINK(ZSTR_VAL(temp_file_path));`
	`2367`	`+ VCWD_UNLINK(ZSTR_VAL(temp_file_path), ZSTR_LEN(temp_file_path));`
`2368`	`2368`	`}`
`2369`	`2369`	`}`
`2370`	`2370`
Original file line number	Diff line number	Diff line change
`@@ -2393,7 +2393,7 @@ PHP_FUNCTION(move_uploaded_file)`
`2393`	`2393`	`}`
`2394`	`2394`	`#endif`
`2395`	`2395`	`} else if (php_copy_file_ex(ZSTR_VAL(path), new_path, STREAM_DISABLE_OPEN_BASEDIR) == SUCCESS) {`
`2396`		`- VCWD_UNLINK(ZSTR_VAL(path));`
	`2396`	`+ VCWD_UNLINK(ZSTR_VAL(path), ZSTR_LEN(path));`
`2397`	`2397`	`successful = 1;`
`2398`	`2398`	`}`
`2399`	`2399`
Original file line number	Diff line number	Diff line change
`@@ -509,7 +509,7 @@ static int php_stdiop_close(php_stream *stream, int close_handle)`
`509`	`509`	`}`
`510`	`510`	`if (data->temp_name) {`
`511`	`511`	`#ifdef PHP_WIN32`
`512`		`- php_win32_ioutil_unlink(ZSTR_VAL(data->temp_name));`
	`512`	`+ php_win32_ioutil_unlink(ZSTR_VAL(data->temp_name), ZSTR_LEN(data->temp_name));`
`513`	`513`	`#else`
`514`	`514`	`unlink(ZSTR_VAL(data->temp_name));`
`515`	`515`	`#endif`
`@@ -1242,8 +1242,6 @@ static int php_plain_files_url_stater(php_stream_wrapper wrapper, const char u`
`1242`	`1242`
`1243`	`1243`	`static int php_plain_files_unlink(php_stream_wrapper wrapper, const char url, int options, php_stream_context *context)`
`1244`	`1244`	`{`
`1245`		`- int ret;`
`1246`		`-`
`1247`	`1245`	`if (strncasecmp(url, "file://", sizeof("file://") - 1) == 0) {`
`1248`	`1246`	`url += sizeof("file://") - 1;`
`1249`	`1247`	`}`
`@@ -1252,8 +1250,9 @@ static int php_plain_files_unlink(php_stream_wrapper wrapper, const char url,`
`1252`	`1250`	`return 0;`
`1253`	`1251`	`}`
`1254`	`1252`
`1255`		`- ret = VCWD_UNLINK(url);`
`1256`		`- if (ret == -1) {`
	`1253`	`+ size_t url_len = strlen(url);`
	`1254`	`+ zend_result ret = VCWD_UNLINK(url, url_len);`
	`1255`	`+ if (ret == FAILURE) {`
`1257`	`1256`	`if (options & REPORT_ERRORS) {`
`1258`	`1257`	`php_error_docref1(NULL, url, E_WARNING, "%s", strerror(errno));`
`1259`	`1258`	`}`
`@@ -1339,7 +1338,7 @@ static bool php_plain_files_rename(php_stream_wrapper *wrapper, const zend_strin`
`1339`	`1338`	`}`
`1340`	`1339`	`# endif`
`1341`	`1340`	`if (success) {`
`1342`		`- VCWD_UNLINK(url_from_ptr);`
	`1341`	`+ VCWD_UNLINK(url_from_ptr, url_from_len);`
`1343`	`1342`	`}`
`1344`	`1343`	`} else {`
`1345`	`1344`	`php_error_docref2(NULL, ZSTR_VAL(url_from), ZSTR_VAL(url_to), E_WARNING, "%s", strerror(errno));`