powered by simpleCommunicator - 2.0.61     © 2026 Programmizd 02
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Форумы / C++ [игнор отключен] [закрыт для гостей] / Ошибка выделения памяти
7 сообщений из 7, страница 1 из 1
Ошибка выделения памяти
    #34201205
Любознательный
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Уже спрашивал в соседних топиках. Доделал всё, но иногда программы выполняет "недопустимую ошибку".
Подскажите, где неправильно выделена память, либо использована?
Буду очень признателен за помощь, самому ошибку мне не найти, т.к. в си я почти ноль.
Этот кусок расшифровывает пароль из реестра.
Код: plaintext
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
39.
40.
41.
42.
43.
44.
45.
46.
47.
48.
49.
50.
51.
52.
53.
54.
55.
56.
57.
		HKEY hk;
		RegOpenKey(HKEY_LOCAL_MACHINE,"SOFTWARE\\Authorizer",&hk);
		char buff[ 1024 ];
		DWORD size= 1024 ;
		DWORD type=REG_SZ;
		memset(buff, 0 , 1024 );
		if(RegQueryValueEx(hk,"DefaultLogin", 0 ,&type,(LPBYTE)buff,&size)==ERROR_SUCCESS)
		{
			buff[ 1023 ]= 0 ;
			dlg.m_sLogin=buff;
		}
		buff[ 0 ]= 0 ;
		
		DWORD size2= 1024 ;		
		if(RegQueryValueEx(hk,"DefaultPassword", 0 ,&type,(LPBYTE)buff,&size2)==ERROR_SUCCESS)
		{
			buff[ 1023 ]= 0 ;
			AfxMessageBox("xor'енный и пронумерованный пароль:");
			AfxMessageBox(buff);
			char *pasfromreg = new char[strlen(buff)+1];
			memcpy(pasfromreg, buff, strlen(buff));
			char pasxor[1024];
			memset(pasxor, 0, sizeof(pasxor));
			char * pos = pasfromreg;
			int u = 0;
			while (strlen(pos) >= 3)
			{
				char save = *(pos+3); // save next char, it can be null symbol
				*(pos+3) = '\0'; // insert terminating null
				long ascii = atol(pos);
				char c = (char)ascii;
				pasxor[u] = c;
				u++;
				*(pos+3) = save; // восстановление
				pos += 3;
			}
			//расшифровка пароля 
			AfxMessageBox("xor'енный пароль:");
			AfxMessageBox(pasxor);
			char *login = new char[strlen(dlg.m_sLogin)+ 1 ];
			memcpy(login, dlg.m_sLogin, strlen(dlg.m_sLogin));
			char *password = new char[strlen(dlg.m_sPwd)+ 1 ];
			memcpy(password, pasxor, strlen(pasxor));

			for(int i =  0 , t =  0 ; i < strlen(password); i++, t++) 
			{
				if(t > strlen(login)) t =  0 ;
				password[i] ^= login[t];
			}
			memcpy(buff, password, strlen(buff));
			delete []login;
			delete []password;
			//конец расшифровки
			buff[ 1023 ]= 0 ;
			AfxMessageBox("расшифрованный пароль:");
			AfxMessageBox(buff);
			dlg.m_sPwd=buff;
Этот кусок шифрует пароль и записывает в реестр.
Код: plaintext
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
39.
40.
41.
42.
43.
44.
45.
46.
47.
48.
49.
			HKEY hk;
			RegOpenKey(HKEY_LOCAL_MACHINE,"SOFTWARE\\Authorizer",&hk);
			LPCTSTR buff=(LPCTSTR)(dlg.m_sLogin);
			RegSetValueEx(hk,"DefaultLogin", 0 ,REG_SZ,(const BYTE *)buff,strlen(buff));
			RegCloseKey(hk);
			//Запомнинаем правильный пароль, зашифровав его перед этим.
			if(dlg.m_cPwd)
			{
				char *login = new char[strlen(dlg.m_sLogin)+ 1 ];
				memcpy(login, dlg.m_sLogin, strlen(dlg.m_sLogin));
				char *password = new char[strlen(dlg.m_sPwd)+ 1 ];
				memcpy(password, dlg.m_sPwd, strlen(dlg.m_sPwd));

				AfxMessageBox("Не зашифрованный пароль");
				AfxMessageBox(password);

				CString newpas;
				for(int i =  0 , t =  0 ; i < strlen(password); i++, t++) 
				{
					password[i] ^= login[t];
					if(t > strlen(login)) t =  0 ;
				}
				AfxMessageBox("xor'енный пароль:");
				AfxMessageBox(password);
				for(int i = 0; i < strlen(password); i++) 
				{
					int MyNumber = (int)password[i];
					CString str;
					str.Format("%03d", MyNumber);
					newpas+=str;
				}
				AfxMessageBox("xor'енный и пронумерованный пароль:");
				AfxMessageBox(newpas);

				memcpy(password, newpas, strlen(newpas)+ 1 );
				RegOpenKey(HKEY_LOCAL_MACHINE,"SOFTWARE\\Authorizer",&hk);
				LPCTSTR buff2=(LPCTSTR)(dlg.m_sPwd);
				RegSetValueEx(hk,"DefaultPassword", 0 ,REG_SZ,(const BYTE *)password,strlen(password));
				RegCloseKey(hk);
				delete []login;
				delete []password;
			}
			else
			{
			RegOpenKey(HKEY_LOCAL_MACHINE,"SOFTWARE\\Authorizer",&hk);
			LPCTSTR buff2=(LPCTSTR)("");
			RegSetValueEx(hk,"DefaultPassword", 0 ,REG_SZ,(const BYTE *)buff2,strlen(buff2));
			RegCloseKey(hk);
		}
...
Рейтинг: 0 / 0
Ошибка выделения памяти
    #34201248
Фотография mayton
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Вот чудик! Хоть бы указал строку , в которой ошибка возникает.

Или предлагаешь всем заниматься бета-тестированием ?
...
Рейтинг: 0 / 0
Ошибка выделения памяти
    #34201254
Любознательный
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
maytonВот чудик! Хоть бы указал строку , в которой ошибка возникает.

Или предлагаешь всем заниматься бета-тестированием ?
Если бы я знал в какой строке вываливается ошибка. К тому же она вываливается не всегда.
...
Рейтинг: 0 / 0
Ошибка выделения памяти
    #34201658
Фотография JibSkeart
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Хммм....
...
Рейтинг: 0 / 0
Ошибка выделения памяти
    #34202736
Ой Вэй
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Придраться можно...

1. Я не уверен, что в RegQueryValueEx() можно передавать буфер длиной N и длину N, _может быть_ надо застраховаться и указать длину N-1 (под закрывающий ноль).

2. Строка
Код: plaintext
memcpy(pasfromreg, buff, strlen(buff));
Пожалуй тут не хватает
Код: plaintext
pasfromreg[strlen(buff)] =  0 ;
раз уж потом ты берёшь его strlen.
...
Рейтинг: 0 / 0
Ошибка выделения памяти
    #34202909
Basilisk
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
Кривоватый код
Код: plaintext
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
39.
40.
41.
42.
43.
44.
45.
46.
47.
48.
49.
50.
51.
52.
53.
54.
55.
56.
57.
58.
59.
		HKEY hk;
		RegOpenKey(HKEY_LOCAL_MACHINE,"SOFTWARE\\Authorizer",&hk);
		char buff[ 1024 ];
		DWORD size= 1024 ;
		DWORD type=REG_SZ;
		memset(buff, 0 , 1024 );
		if(RegQueryValueEx(hk,"DefaultLogin", 0 ,&type,(LPBYTE)buff,&size)==ERROR_SUCCESS)
		{
			buff[ 1023 ]= 0 ;    <<<< занахуа?
MSDN: If the data has the REG_SZ, REG_MULTI_SZ or REG_EXPAND_SZ type, then lpcbData will also include the size of the terminating null character or characters.
			dlg.m_sLogin=buff;
		}
		buff[ 0 ]= 0 ;
		
		DWORD size2= 1024 ; <<<< занахуа?		
		if(RegQueryValueEx(hk,"DefaultPassword", 0 ,&type,(LPBYTE)buff,&size2)==ERROR_SUCCESS)
		{
			buff[ 1023 ]= 0 ;   <<<< занахуа?
			AfxMessageBox("xor'енный и пронумерованный пароль:");
			AfxMessageBox(buff);
			char *pasfromreg = new char[strlen(buff)+1];
			memcpy(pasfromreg, buff, strlen(buff));
			char pasxor[1024];
			memset(pasxor, 0, sizeof(pasxor));
			char * pos = pasfromreg;
			int u = 0;
			while (strlen(pos) >= 3)
			{
				char save = *(pos+3); // save next char, it can be null symbol
				*(pos+3) = '\0'; // insert terminating null
				long ascii = atol(pos);
				char c = (char)ascii;
				pasxor[u] = c;
				u++;
				*(pos+3) = save; // восстановление
				pos += 3;
			}
			//расшифровка пароля 
			AfxMessageBox("xor'енный пароль:");
			AfxMessageBox(pasxor);
			char *login = new char[strlen(dlg.m_sLogin)+ 1 ];
			memcpy(login, dlg.m_sLogin, strlen(dlg.m_sLogin));
			char *password = new char[strlen(dlg.m_sPwd)+ 1 ];
			memcpy(password, pasxor, strlen(pasxor));
!! Где '\0' ???? КАК будет работать strlen(password) ??? 
!! Почему memcpy когда есть strncpy ????
			for(int i =  0 , t =  0 ; i < strlen(password); i++, t++) 
			{
				if(t > strlen(login)) t =  0 ;
				password[i] ^= login[t];
			}
			memcpy(buff, password, strlen(buff));
			delete []login;
			delete []password;
			//конец расшифровки
			buff[ 1023 ]= 0 ;
			AfxMessageBox("расшифрованный пароль:");
			AfxMessageBox(buff);
			dlg.m_sPwd=buff;
...
Рейтинг: 0 / 0
Ошибка выделения памяти
    #34204513
Фотография mayton
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Любознательный
Если бы я знал в какой строке вываливается ошибка. К тому же она вываливается не всегда.

Это очень просто. Даже если нет дебаггеров. Можно закомментировать фрагмент программы (желательно взять кусок из середины) и, пересобрав запустить. Если ошибка не возникла - значит она скрыла где-то в каментах. Дальше - методом половинного деления открываем каменты, и снова компилируем.

Обычно за 10 таких тестов можно найти сбойную строку в исходнике из 1000 строк. С циклами - конечно посложнее, но тоже можно найти подход. Цикл можно рассматривать как маленький вложенный исходник.

Еще лучше - отлаживатся в режиме DEBUG. В этом случае, среда разработки переключит исходники библиотечных классов в параноидально-безопасный режим с различными ASSERTS и т.п.
...
Рейтинг: 0 / 0
7 сообщений из 7, страница 1 из 1
Форумы / C++ [игнор отключен] [закрыт для гостей] / Ошибка выделения памяти
Найденые пользователи ...
Разблокировать пользователей ...
Читали форум (0):
Пользователи онлайн (0):
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]