# HG changeset patch # User Christian Brabandt # Date 1457643605 -3600 # Node ID 7376d36395f0f511bc8837447446f235f18b0ae4 # Parent b0e1295c1edf83ea8defa18ac4b91d6c012dcdc3 commit https://github.com/vim/vim/commit/7bffaa9f9b477969d85cef41adeadc4506373708 Author: Bram Moolenaar Date: Thu Mar 10 21:46:03 2016 +0100 patch 7.4.1532 Problem: MS-Windows channel leaks file descriptor. Solution: Use CreateFile with the right options. (Yasuhiro Matsumoto) diff --git a/src/os_win32.c b/src/os_win32.c --- a/src/os_win32.c +++ b/src/os_win32.c @@ -4992,6 +4992,41 @@ mch_call_shell( } #if defined(FEAT_JOB) || defined(PROTO) + static HANDLE +job_io_file_open( + char_u *fname, + DWORD dwDesiredAccess, + DWORD dwShareMode, + LPSECURITY_ATTRIBUTES lpSecurityAttributes, + DWORD dwCreationDisposition, + DWORD dwFlagsAndAttributes) +{ + HANDLE h; +# ifdef FEAT_MBYTE + WCHAR *wn = NULL; + if (enc_codepage >= 0 && (int)GetACP() != enc_codepage) + { + wn = enc_to_utf16(fname, NULL); + if (wn != NULL) + { + h = CreateFileW(wn, dwDesiredAccess, dwShareMode, + lpSecurityAttributes, dwCreationDisposition, + dwFlagsAndAttributes, NULL); + vim_free(wn); + if (h == INVALID_HANDLE_VALUE + && GetLastError() == ERROR_CALL_NOT_IMPLEMENTED) + wn = NULL; + } + } + if (wn == NULL) +# endif + + h = CreateFile((LPCSTR)fname, dwDesiredAccess, dwShareMode, + lpSecurityAttributes, dwCreationDisposition, + dwFlagsAndAttributes, NULL); + return h; +} + void mch_start_job(char *cmd, job_T *job, jobopt_T *options) { @@ -5046,13 +5081,14 @@ mch_start_job(char *cmd, job_T *job, job { char_u *fname = options->jo_io_name[PART_IN]; - int fd = mch_open((char *)fname, O_RDONLY, 0); - if (fd < 0) + ifd[0] = job_io_file_open(fname, GENERIC_READ, + FILE_SHARE_READ | FILE_SHARE_WRITE, + &saAttr, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL); + if (ifd[0] == INVALID_HANDLE_VALUE) { EMSG2(_(e_notopen), fname); goto failed; } - ifd[0] = (HANDLE)_get_osfhandle(fd); } else if (!use_null_for_in && (!CreatePipe(&ifd[0], &ifd[1], &saAttr, 0) @@ -5063,13 +5099,14 @@ mch_start_job(char *cmd, job_T *job, job { char_u *fname = options->jo_io_name[PART_OUT]; - int fd = mch_open((char *)fname, O_WRONLY | O_CREAT | O_TRUNC, 0644); - if (fd < 0) + ofd[1] = job_io_file_open(fname, GENERIC_WRITE, + FILE_SHARE_READ | FILE_SHARE_WRITE, + &saAttr, CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL); + if (ofd[1] == INVALID_HANDLE_VALUE) { EMSG2(_(e_notopen), fname); goto failed; } - ofd[1] = (HANDLE)_get_osfhandle(fd); } else if (!use_null_for_out && (!CreatePipe(&ofd[0], &ofd[1], &saAttr, 0) @@ -5080,13 +5117,14 @@ mch_start_job(char *cmd, job_T *job, job { char_u *fname = options->jo_io_name[PART_ERR]; - int fd = mch_open((char *)fname, O_WRONLY | O_CREAT | O_TRUNC, 0600); - if (fd < 0) + efd[1] = job_io_file_open(fname, GENERIC_WRITE, + FILE_SHARE_READ | FILE_SHARE_WRITE, + &saAttr, CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL); + if (efd[1] == INVALID_HANDLE_VALUE) { EMSG2(_(e_notopen), fname); goto failed; } - efd[1] = (HANDLE)_get_osfhandle(fd); } else if (!use_out_for_err && !use_null_for_err && (!CreatePipe(&efd[0], &efd[1], &saAttr, 0) diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -744,6 +744,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 1532, +/**/ 1531, /**/ 1530,