diff options
author | Sefa Eyeoglu <contact@scrumplex.net> | 2022-07-16 21:15:55 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-07-16 21:15:55 +0200 |
commit | 8cf42f4e152333eb216d24c33a095d05dc0499d4 (patch) | |
tree | d980111b3245de08687b535f9b49b2b8ba5a125c | |
parent | 04818ca626930fa6723bb73f5d7246a71ed352a1 (diff) | |
parent | 17ea51ce276a1ad639fb7f62724230de25983d4c (diff) | |
download | PrismLauncher-8cf42f4e152333eb216d24c33a095d05dc0499d4.tar.gz PrismLauncher-8cf42f4e152333eb216d24c33a095d05dc0499d4.tar.bz2 PrismLauncher-8cf42f4e152333eb216d24c33a095d05dc0499d4.zip |
Merge pull request #902 from Scrumplex/fix-unicode-windows
-rw-r--r-- | launcher/java/JavaUtils.cpp | 19 |
1 files changed, 7 insertions, 12 deletions
diff --git a/launcher/java/JavaUtils.cpp b/launcher/java/JavaUtils.cpp index c2b776ae..749c9c88 100644 --- a/launcher/java/JavaUtils.cpp +++ b/launcher/java/JavaUtils.cpp @@ -202,14 +202,6 @@ QList<JavaInstallPtr> JavaUtils::FindJavaFromRegistryKey(DWORD keyType, QString { // Read the current type version from the registry. // This will be used to find any key that contains the JavaHome value. - char *value = new char[0]; - DWORD valueSz = 0; - if (RegQueryValueExW(jreKey, L"CurrentVersion", NULL, NULL, (BYTE *)value, &valueSz) == - ERROR_MORE_DATA) - { - value = new char[valueSz]; - RegQueryValueExW(jreKey, L"CurrentVersion", NULL, NULL, (BYTE *)value, &valueSz); - } TCHAR subKeyName[255]; DWORD subKeyNameSize, numSubKeys, retCode; @@ -237,22 +229,25 @@ QList<JavaInstallPtr> JavaUtils::FindJavaFromRegistryKey(DWORD keyType, QString KEY_READ | KEY_WOW64_64KEY, &newKey) == ERROR_SUCCESS) { // Read the JavaHome value to find where Java is installed. - value = new char[0]; - valueSz = 0; + TCHAR *value = NULL; + DWORD valueSz = 0; if (RegQueryValueExW(newKey, keyJavaDir.toStdWString().c_str(), NULL, NULL, (BYTE *)value, &valueSz) == ERROR_MORE_DATA) { - value = new char[valueSz]; + value = new TCHAR[valueSz]; RegQueryValueExW(newKey, keyJavaDir.toStdWString().c_str(), NULL, NULL, (BYTE *)value, &valueSz); + QString newValue = QString::fromWCharArray(value); + delete [] value; + // Now, we construct the version object and add it to the list. JavaInstallPtr javaVersion(new JavaInstall()); javaVersion->id = newSubkeyName; javaVersion->arch = archType; javaVersion->path = - QDir(FS::PathCombine(value, "bin")).absoluteFilePath("javaw.exe"); + QDir(FS::PathCombine(newValue, "bin")).absoluteFilePath("javaw.exe"); javas.append(javaVersion); } |