From: Corinna Vinschen <corinna@vinschen.de>
Date: Thu, 15 Oct 2009 11:40:02 +0000 (+0000)
Subject: 	* cygpath.cc (get_special_folder): Fetch path as WCHAR and convert
X-Git-Tag: sid-snapshot-20091101~17
X-Git-Url: https://sourceware.org/git/?a=commitdiff_plain;h=175e39bb7f7aefab3a315427b564faf1f0b33173;p=newlib-cygwin.git

	* cygpath.cc (get_special_folder): Fetch path as WCHAR and convert
	using my_wcstombs.
	(get_user_folder): Remove.
	(do_sysfolders): Use get_special_folder instead of get_user_folder.
	Fetch system paths as WCHAR and convert using my_wcstombs.
---

diff --git a/winsup/utils/ChangeLog b/winsup/utils/ChangeLog
index f3d05d9b2..00d61bab4 100644
--- a/winsup/utils/ChangeLog
+++ b/winsup/utils/ChangeLog
@@ -1,3 +1,11 @@
+2009-10-15  Corinna Vinschen  <corinna@vinschen.de>
+
+	* cygpath.cc (get_special_folder): Fetch path as WCHAR and convert
+	using my_wcstombs.
+	(get_user_folder): Remove.
+	(do_sysfolders): Use get_special_folder instead of get_user_folder.
+	Fetch system paths as WCHAR and convert using my_wcstombs.
+
 2009-10-12  Corinna Vinschen  <corinna@vinschen.de>
 
 	* kill.cc (main): Skip to PID loop on invalid option to handle
diff --git a/winsup/utils/cygpath.cc b/winsup/utils/cygpath.cc
index a267181c0..83d4e3fd3 100644
--- a/winsup/utils/cygpath.cc
+++ b/winsup/utils/cygpath.cc
@@ -558,45 +558,46 @@ get_mixed_name (const char* filename)
 static bool
 get_special_folder (char* path, int id)
 {
-  path[0] = 0;
+  WCHAR wpath[MAX_PATH];
+
+  path[0] = '\0';
+  wpath[0] = L'\0';
   LPITEMIDLIST pidl = 0;
   if (SHGetSpecialFolderLocation (NULL, id, &pidl) != S_OK)
     return false;
-  if (!SHGetPathFromIDList (pidl, path) || !path[0])
+  if (!SHGetPathFromIDListW (pidl, wpath) || !wpath[0])
     return false;
+  my_wcstombs (path, wpath, PATH_MAX);
   return true;
 }
 
-static void
-get_user_folder (char* path, int id, int allid)
-{
-  get_special_folder (path, allusers_flag ? allid : id);
-}
-
 static void
 do_sysfolders (char option)
 {
-  char *buf, buf1[MAX_PATH], buf2[PATH_MAX];
+  char *buf, buf1[PATH_MAX], buf2[PATH_MAX];
+  WCHAR wbuf[MAX_PATH];
   DWORD len = MAX_PATH;
-  WIN32_FIND_DATA w32_fd;
+  WIN32_FIND_DATAW w32_fd;
   HINSTANCE k32;
-  BOOL (*GetProfilesDirectoryAPtr) (LPSTR, LPDWORD) = 0;
+  BOOL (*GetProfilesDirectoryAPtrW) (LPWSTR, LPDWORD) = 0;
 
   buf = buf1;
   buf[0] = 0;
   switch (option)
     {
     case 'D':
-      get_user_folder (buf, CSIDL_DESKTOPDIRECTORY,
-			    CSIDL_COMMON_DESKTOPDIRECTORY);
+      get_special_folder (buf, allusers_flag ? CSIDL_COMMON_DESKTOPDIRECTORY
+					     : CSIDL_DESKTOPDIRECTORY);
       break;
 
     case 'P':
-      get_user_folder (buf, CSIDL_PROGRAMS, CSIDL_COMMON_PROGRAMS);
+      get_special_folder (buf, allusers_flag ? CSIDL_COMMON_PROGRAMS
+					     : CSIDL_PROGRAMS);
       break;
 
     case 'O':
-      get_user_folder (buf, CSIDL_PERSONAL, CSIDL_COMMON_DOCUMENTS);
+      get_special_folder (buf, allusers_flag ? CSIDL_COMMON_DOCUMENTS
+					     : CSIDL_PERSONAL);
       break;
 
     case 'F':
@@ -616,25 +617,35 @@ do_sysfolders (char option)
     case 'H':
       k32 = LoadLibrary ("userenv");
       if (k32)
-	GetProfilesDirectoryAPtr = (BOOL (*) (LPSTR, LPDWORD))
-	  GetProcAddress (k32, "GetProfilesDirectoryA");
-      if (GetProfilesDirectoryAPtr)
-	(*GetProfilesDirectoryAPtr) (buf, &len);
+	GetProfilesDirectoryAPtrW = (BOOL (*) (LPWSTR, LPDWORD))
+	  GetProcAddress (k32, "GetProfilesDirectoryW");
+      if (GetProfilesDirectoryAPtrW)
+	(*GetProfilesDirectoryAPtrW) (wbuf, &len);
       else
 	{
-	  GetWindowsDirectory (buf, MAX_PATH);
-	  strcat (buf, "\\Profiles");
+	  GetWindowsDirectoryW (wbuf, MAX_PATH);
+	  wcscat (wbuf, L"\\Profiles");
 	}
+      my_wcstombs (buf, wbuf, PATH_MAX);
       break;
 
     case 'S':
-      GetSystemDirectory (buf, MAX_PATH);
-      FindFirstFile (buf, &w32_fd);
-      strcpy (strrchr (buf, '\\') + 1, w32_fd.cFileName);
+      {
+	HANDLE fh;
+
+	GetSystemDirectoryW (wbuf, MAX_PATH);
+	if ((fh = FindFirstFileW (wbuf, &w32_fd)) != INVALID_HANDLE_VALUE)
+	  {
+	    FindClose (fh);
+	    wcscpy (wcsrchr (wbuf, L'\\') + 1, w32_fd.cFileName);
+	  }
+	my_wcstombs (buf, wbuf, PATH_MAX);
+      }
       break;
 
     case 'W':
-      GetWindowsDirectory (buf, MAX_PATH);
+      GetWindowsDirectoryW (wbuf, MAX_PATH);
+      my_wcstombs (buf, wbuf, PATH_MAX);
       break;
 
     default: