Changeset 34547 in vbox for trunk/src/VBox/Installer/win/InstallHelper

Timestamp:

Dec 1, 2010 9:08:55 AM (14 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

68338

Message:

Windows installer: fixed misinterpretation of the sizeOfBuffer parameter of _stprintf_s => swprintf_s

File:

• trunk/src/VBox/Installer/win/InstallHelper/VBoxInstallHelper.cpp (modified) (9 diffs)

trunk/src/VBox/Installer/win/InstallHelper/VBoxInstallHelper.cpp

@@ -193 +193 @@
             break;
 
-        _stprintf_s(szPath, sizeof(szPath), L"%s\\InstallPath", szRoot);
+        _stprintf_s(szPath, sizeof(szPath) / sizeof(TCHAR), L"%s\\InstallPath", szRoot);
         dwLen = sizeof(szVal);
 
@@ -216 +216 @@
         /* Cool, check for installed Win32 extensions. */
         LogString(hModule, TEXT("InstallPythonAPI: Python installed. Checking for Win32 extensions ..."));
-        _stprintf_s(szExec, sizeof(szExec), L"%s\\python.exe", szVal);
-        _stprintf_s(szCmdLine, sizeof(szCmdLine), L"%s\\python.exe -c \"import win32api\"", szVal);
+        _stprintf_s(szExec, sizeof(szExec) / sizeof(TCHAR), L"%s\\python.exe", szVal);
+        _stprintf_s(szCmdLine, sizeof(szCmdLine) / sizeof(TCHAR), L"%s\\python.exe -c \"import win32api\"", szVal);
 
         if (   (0 == Exec(hModule, szExec, szCmdLine, NULL, &dwExitCode))
@@ -236 +236 @@
 
         /* Set final path. */
-        _stprintf_s(szPath, sizeof(szPath), L"%s\\sdk\\install", szVBoxAPISetupPath);
+        _stprintf_s(szPath, sizeof(szPath) / sizeof(TCHAR), L"%s\\sdk\\install", szVBoxAPISetupPath);
 
         /* Install our API module. */
-        _stprintf_s(szCmdLine, sizeof(szCmdLine), L"%s\\python.exe vboxapisetup.py install", szVal);
+        _stprintf_s(szCmdLine, sizeof(szCmdLine) / sizeof(TCHAR), L"%s\\python.exe vboxapisetup.py install", szVal);
 
         /* Set required environment variables. */
@@ -278 +278 @@
 
         if (wcsicmp(L"General", pszSection) != 0)
-            _stprintf_s(szKey, sizeof(szKey), L"SOFTWARE\\%s\\VirtualBox\\Branding\\", VBOX_VENDOR_SHORT, pszSection);
+            _stprintf_s(szKey, sizeof(szKey) / sizeof(TCHAR), L"SOFTWARE\\%s\\VirtualBox\\Branding\\", VBOX_VENDOR_SHORT, pszSection);
         else
-            _stprintf_s(szKey, sizeof(szKey), L"SOFTWARE\\%s\\VirtualBox\\Branding", VBOX_VENDOR_SHORT);
+            _stprintf_s(szKey, sizeof(szKey) / sizeof(TCHAR), L"SOFTWARE\\%s\\VirtualBox\\Branding", VBOX_VENDOR_SHORT);
 
         rc = RegOpenKeyEx(HKEY_LOCAL_MACHINE, szKey,
@@ -308 +308 @@
     TCHAR szSource[_MAX_PATH + 1];
 
-    _stprintf_s(szDest, sizeof(szDest), L"%s%c", pszDestDir, '\0');
-    _stprintf_s(szSource, sizeof(szSource), L"%s%c", pszSourceDir, '\0');
+    _stprintf_s(szDest, sizeof(szDest) / sizeof(TCHAR), L"%s%c", pszDestDir, '\0');
+    _stprintf_s(szSource, sizeof(szSource) / sizeof(TCHAR), L"%s%c", pszSourceDir, '\0');
 
     SHFILEOPSTRUCT s = {0};
@@ -339 +339 @@
     TCHAR szDest[_MAX_PATH + 1];
 
-    _stprintf_s(szDest, sizeof(szDest), L"%s%c", pszDestDir, '\0');
+    _stprintf_s(szDest, sizeof(szDest) / sizeof(TCHAR), L"%s%c", pszDestDir, '\0');
 
     SHFILEOPSTRUCT s = {0};
@@ -368 +368 @@
     TCHAR szSource[_MAX_PATH + 1];
 
-    _stprintf_s(szDest, sizeof(szDest), L"%s%c", pszDestDir, '\0');
-    _stprintf_s(szSource, sizeof(szSource), L"%s%c", pszSourceDir, '\0');
+    _stprintf_s(szDest, sizeof(szDest) / sizeof(TCHAR), L"%s%c", pszDestDir, '\0');
+    _stprintf_s(szSource, sizeof(szSource) / sizeof(TCHAR), L"%s%c", pszSourceDir, '\0');
 
     SHFILEOPSTRUCT s = {0};
@@ -406 +406 @@
     {
         /** @todo Check trailing slash after %s. */
-        _stprintf_s(szPathDest, sizeof(szPathDest), L"%scustom", szPathTargetDir);
+        _stprintf_s(szPathDest, sizeof(szPathDest) / sizeof(TCHAR), L"%scustom", szPathTargetDir);
         rc = RemoveDir(hModule, szPathDest);
         if (rc != ERROR_SUCCESS)
         {
             /* Check for hidden .custom directory and remove it. */
-            _stprintf_s(szPathDest, sizeof(szPathDest), L"%s.custom", szPathTargetDir);
+            _stprintf_s(szPathDest, sizeof(szPathDest) / sizeof(TCHAR), L"%s.custom", szPathTargetDir);
             rc = RemoveDir(hModule, szPathDest);
         }
@@ -436 +436 @@
     {
         /** @todo Check for trailing slash after %s. */
-        _stprintf_s(szPathDest, sizeof(szPathDest), L"%s", szPathTargetDir);
-        _stprintf_s(szPathSource, sizeof(szPathSource), L"%s.custom", szPathMSI);
+        _stprintf_s(szPathDest, sizeof(szPathDest) / sizeof(TCHAR), L"%s", szPathTargetDir);
+        _stprintf_s(szPathSource, sizeof(szPathSource) / sizeof(TCHAR), L"%s.custom", szPathMSI);
         rc = CopyDir(hModule, szPathDest, szPathSource);
         if (rc == ERROR_SUCCESS)
         {
-            _stprintf_s(szPathDest, sizeof(szPathDest), L"%scustom", szPathTargetDir);
-            _stprintf_s(szPathSource, sizeof(szPathSource), L"%s.custom", szPathTargetDir);
+            _stprintf_s(szPathDest, sizeof(szPathDest) / sizeof(TCHAR), L"%scustom", szPathTargetDir);
+            _stprintf_s(szPathSource, sizeof(szPathSource) / sizeof(TCHAR), L"%s.custom", szPathTargetDir);
             rc = RenameDir(hModule, szPathDest, szPathSource);
         }