8b9a1c
To: vim_dev@googlegroups.com
8b9a1c
Subject: Patch 7.4.013
8b9a1c
Fcc: outbox
8b9a1c
From: Bram Moolenaar <Bram@moolenaar.net>
8b9a1c
Mime-Version: 1.0
8b9a1c
Content-Type: text/plain; charset=UTF-8
8b9a1c
Content-Transfer-Encoding: 8bit
8b9a1c
------------
8b9a1c
8b9a1c
Patch 7.4.013
8b9a1c
Problem:    File name buffer too small for utf-8.
8b9a1c
Solution:   Use character count instead of byte count. (Ken Takata)
8b9a1c
Files:	    src/os_mswin.c
8b9a1c
8b9a1c
8b9a1c
*** ../vim-7.4.012/src/os_mswin.c	2013-08-30 16:44:15.000000000 +0200
8b9a1c
--- src/os_mswin.c	2013-08-30 16:47:54.000000000 +0200
8b9a1c
***************
8b9a1c
*** 456,462 ****
8b9a1c
--- 456,469 ----
8b9a1c
      int
8b9a1c
  mch_isFullName(char_u *fname)
8b9a1c
  {
8b9a1c
+ #ifdef FEAT_MBYTE
8b9a1c
+     /* WinNT and later can use _MAX_PATH wide characters for a pathname, which
8b9a1c
+      * means that the maximum pathname is _MAX_PATH * 3 bytes when 'enc' is
8b9a1c
+      * UTF-8. */
8b9a1c
+     char szName[_MAX_PATH * 3 + 1];
8b9a1c
+ #else
8b9a1c
      char szName[_MAX_PATH + 1];
8b9a1c
+ #endif
8b9a1c
  
8b9a1c
      /* A name like "d:/foo" and "//server/share" is absolute */
8b9a1c
      if ((fname[0] && fname[1] == ':' && (fname[2] == '/' || fname[2] == '\\'))
8b9a1c
***************
8b9a1c
*** 464,470 ****
8b9a1c
  	return TRUE;
8b9a1c
  
8b9a1c
      /* A name that can't be made absolute probably isn't absolute. */
8b9a1c
!     if (mch_FullName(fname, szName, _MAX_PATH, FALSE) == FAIL)
8b9a1c
  	return FALSE;
8b9a1c
  
8b9a1c
      return pathcmp(fname, szName, -1) == 0;
8b9a1c
--- 471,477 ----
8b9a1c
  	return TRUE;
8b9a1c
  
8b9a1c
      /* A name that can't be made absolute probably isn't absolute. */
8b9a1c
!     if (mch_FullName(fname, szName, sizeof(szName) - 1, FALSE) == FAIL)
8b9a1c
  	return FALSE;
8b9a1c
  
8b9a1c
      return pathcmp(fname, szName, -1) == 0;
8b9a1c
***************
8b9a1c
*** 498,507 ****
8b9a1c
      int
8b9a1c
  vim_stat(const char *name, struct stat *stp)
8b9a1c
  {
8b9a1c
      char	buf[_MAX_PATH + 1];
8b9a1c
      char	*p;
8b9a1c
  
8b9a1c
!     vim_strncpy((char_u *)buf, (char_u *)name, _MAX_PATH);
8b9a1c
      p = buf + strlen(buf);
8b9a1c
      if (p > buf)
8b9a1c
  	mb_ptr_back(buf, p);
8b9a1c
--- 505,521 ----
8b9a1c
      int
8b9a1c
  vim_stat(const char *name, struct stat *stp)
8b9a1c
  {
8b9a1c
+ #ifdef FEAT_MBYTE
8b9a1c
+     /* WinNT and later can use _MAX_PATH wide characters for a pathname, which
8b9a1c
+      * means that the maximum pathname is _MAX_PATH * 3 bytes when 'enc' is
8b9a1c
+      * UTF-8. */
8b9a1c
+     char	buf[_MAX_PATH * 3 + 1];
8b9a1c
+ #else
8b9a1c
      char	buf[_MAX_PATH + 1];
8b9a1c
+ #endif
8b9a1c
      char	*p;
8b9a1c
  
8b9a1c
!     vim_strncpy((char_u *)buf, (char_u *)name, sizeof(buf) - 1);
8b9a1c
      p = buf + strlen(buf);
8b9a1c
      if (p > buf)
8b9a1c
  	mb_ptr_back(buf, p);
8b9a1c
*** ../vim-7.4.012/src/version.c	2013-08-30 16:44:15.000000000 +0200
8b9a1c
--- src/version.c	2013-08-30 16:47:36.000000000 +0200
8b9a1c
***************
8b9a1c
*** 740,741 ****
8b9a1c
--- 740,743 ----
8b9a1c
  {   /* Add new patch number below this line */
8b9a1c
+ /**/
8b9a1c
+     13,
8b9a1c
  /**/
8b9a1c
8b9a1c
-- 
8b9a1c
hundred-and-one symptoms of being an internet addict:
8b9a1c
143. You dream in pallettes of 216 websafe colors.
8b9a1c
8b9a1c
 /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
8b9a1c
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
8b9a1c
\\\  an exciting new programming language -- http://www.Zimbu.org        ///
8b9a1c
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///