]> granicus.if.org Git - php/commitdiff
Fix parameter order
authorChristoph M. Becker <cmbecker69@gmx.de>
Mon, 22 Apr 2019 09:46:19 +0000 (11:46 +0200)
committerChristoph M. Becker <cmbecker69@gmx.de>
Mon, 22 Apr 2019 10:38:24 +0000 (12:38 +0200)
The parameter order of `php_sys_link()` should be identical on Windows
and POSIX, bug commit c9861bd[1] got that wrong.

[1] <http://git.php.net/?p=php-src.git;a=commit;h=c9861bd7a974ef1555395b879c1035a0bbc12ef1>

ext/standard/link_win32.c
win32/ioutil.c

index e8f9e5f8524b8881cea3b9558661e6696dc62ab5..124be74dc4e06117888f38c4b2079f84d3509280 100644 (file)
@@ -184,9 +184,7 @@ PHP_FUNCTION(link)
        char source_p[MAXPATHLEN];
        char dest_p[MAXPATHLEN];
 
-       /*First argument to link function is the target and hence should go to frompath
-         Second argument to link function is the link itself and hence should go to topath */
-       if (zend_parse_parameters(ZEND_NUM_ARGS(), "ss", &frompath, &frompath_len, &topath, &topath_len) == FAILURE) {
+       if (zend_parse_parameters(ZEND_NUM_ARGS(), "ss", &topath, &topath_len, &frompath, &frompath_len) == FAILURE) {
                return;
        }
 
index c819e417128159a007dca90ab0a60775382cc189..7f8948192e916b76d58602685b0c1803a291ab2b 100644 (file)
@@ -832,7 +832,7 @@ PW32IO int php_win32_ioutil_link_w(const wchar_t *target, const wchar_t *link)
 {/*{{{*/
        BOOL res;
 
-       res = CreateHardLinkW(target, link, NULL);
+       res = CreateHardLinkW(link, target, NULL);
        if (!res) {
                SET_ERRNO_FROM_WIN32_CODE(GetLastError());
                return -1;