Почему данные не записываются в реестр?

Question

[Ternick]

Я компилирую приложение как x86!
Функция:

bool addToAutorun(string path) {
	HKEY hKey;
	LONG error = RegCreateKeyEx(HKEY_LOCAL_MACHINE, (LPCSTR)"SOFTWARE\\WOW6432Node\\Microsoft\\Windows\\CurrentVersion\\Run", NULL, NULL, REG_OPTION_NON_VOLATILE, KEY_WOW64_32KEY, NULL, &hKey, NULL);
	if (hKey) {
		RegSetValueA(hKey, "test", REG_SZ, (LPCSTR)path.c_str(), sizeof(path.c_str()));
		RegCloseKey(hKey);
		return TRUE;
	}
	else {
		RegCloseKey(hKey);
		cout << error << endl;
		return FALSE;
	}
}

Я пробовал KEY_WOW64_32KEY этот ключ и KEY_SET_VALUE.
ААА, ЧТО ТУТ МОЖЕТ БЫТЬ НЕ ТАК ?
Я запускаю от администратора приложение.
Ошибок нет, но и результатов нет !

Answer 1

[jcmvbkbc]

LONG error = RegCreateKeyEx(...);
  if (hKey) {

Ошибок нет, но и результатов нет !

Ошибок нет, потому что ты не проверяешь код ошибки, а вместо этого проверяешь hKey.
Если внимательно прочитать https://docs.microsoft.com/en-us/windows/win32/sys... то можно увидеть для KEY_WOW64_32KEY, что

This flag must be combined using the OR operator with the other flags in this table that either query or access registry values.

Подозреваю, что там должно быть KEY_WOW64_32KEY | KEY_SET_VALUE.

RegSetValueA(hKey, "test", REG_SZ, (LPCSTR)path.c_str(), sizeof(path.c_str()));

Не sizeof. И код ошибки тоже не проверяешь.

Comments

[Ternick]

strlen ?

[Ternick]

А как проверить то ?

[jcmvbkbc]

А как проверить то ?

Ternick,
If the function succeeds, the return value is ERROR_SUCCESS.

[jcmvbkbc]

strlen ?

Ternick, path.size() + 1?

[Ternick]

jcmvbkbc, А любая функция из Windows.h можно так поверить ?
Ещё слышал что вроде как SUCCEEDED тоже значит ?

[jcmvbkbc]

А любая функция из Windows.h можно так поверить ?

Ternick, на любую функцию нужно сначала прочитать документацию. Там должно быть написано.

Ещё слышал что вроде как

Вот тебе откровение.

[Ternick]

А теперь та что ни так ?

bool addToAutorun(string path) {
	HKEY hKey;
	if (RegCreateKeyEx(HKEY_LOCAL_MACHINE, (LPCSTR)"SOFTWARE\\WOW6432Node\\Microsoft\\Windows\\CurrentVersion\\Run", NULL, NULL, REG_OPTION_NON_VOLATILE, KEY_WOW64_32KEY | KEY_SET_VALUE, NULL, &hKey, NULL) == ERROR_SUCCESS) {
		if (RegSetValueA(hKey, "test", REG_SZ, (LPCSTR)path.c_str(), path.size() + 1) == ERROR_SUCCESS) {
			RegCloseKey(hKey);
			return TRUE;
		}
		else {
			RegCloseKey(hKey);
			return FALSE;
		}
	}
	else {
		RegCloseKey(hKey);
		return FALSE;
	}

Как запускаю от админа функция возвращает TRUE т.е. 1.

[jcmvbkbc]

Ternick, а зачем ты лезешь под WOW6432Node? Вроде же этот ключ называется просто SOFTWARE\Microsoft\Windows\CurrentVersion\Run?

[Ternick]

jcmvbkbc, Потому что компилирую под x86, а машина то x64 и по моему как то так оно и должно работать.

[Ternick]

jcmvbkbc, Оно даже так не работает, хнык хнык :(
ПРИЧЁМ ВОЗВРАЩАЕТ ТО 1!

[jcmvbkbc]

Оно даже так не работает,

Ternick, ты в реестр-то смотрел?

[Ternick]

jcmvbkbc, ДА, Я КАЖДЫЙ РАЗ МОНИТОРЮ ИЗМЕНЕНИЯ В РЕЕСТРЕ !
Нашёл после каких то тестов 2 папки, но как я их создал я уже не понял.

[jcmvbkbc]

и по моему как то так оно и должно

Ternick, я тебе настоятельно рекомендую прочитать документацию на фичи которые ты пытаешься использовать.
Подозреваю, что после этого ты выкинешь упоминания WOW6432Node и KEY_WOW64_32KEY и всё у тебя найдётся.

[Ternick]

jcmvbkbc, НЕТ ЭТО ТАК ПОЧЕМУ - ТО НЕ РАБОТАЕТ !

bool addToAutorun(string path) {
	HKEY hKey;
	if (RegCreateKeyExA(HKEY_LOCAL_MACHINE, (LPCSTR)"SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Run\\", NULL, NULL, REG_OPTION_NON_VOLATILE, KEY_ALL_ACCESS, NULL, &hKey, NULL) == ERROR_SUCCESS) {
		if (RegSetValueA(hKey, "test", REG_SZ, (LPCSTR)path.c_str(), path.size() + 1) == ERROR_SUCCESS) {
			RegCloseKey(hKey);
			return TRUE;
		}
		else {
			RegCloseKey(hKey);
			return FALSE;
		}
	}
	else {
		RegCloseKey(hKey);
		return FALSE;
	}
}

И ВСЕ РАВНО НЕ РАБОТАЕТ, ТА ПОЧЕМУ ?

[jcmvbkbc]

И ВСЕ РАВНО НЕ РАБОТАЕТ, ТА ПОЧЕМУ ?

Ternick, хм, интересно, содержимое реестра с точки зрения 32-битного и 64-битного приложений разное. Наверно читать тут: https://docs.microsoft.com/en-us/windows/win32/win...

[jcmvbkbc]

Я КАЖДЫЙ РАЗ МОНИТОРЮ ИЗМЕНЕНИЯ В РЕЕСТРЕ

Ternick, полагаю, 64-битной программой? Тогда надо указать KEY_WOW64_64KEY, а путь в реестре -- SOFTWARE\Microsoft\Windows\CurrentVersion\Run

[Ternick]

jcmvbkbc, Я разобрался с этой проблемой и в этот раз не скажу ответ на эту задачку :)
Я говорил что система x64, а программа x86, хотя я уже не самом деле не совсем понимаю как это работает, но не важно. Спасибо за советы интересные :)
Всё что хочу если кто то окажется в моей ситуации могу сказать только одно, обратите внимание на своеобразное название функций :)

[jcmvbkbc]

Я разобрался с этой проблемой и в этот раз не скажу ответ на эту задачку :)

Ternick, ок, я на твои вопросы больше не буду отвечать.