Гость
Войти | Профиль | Очистить
Целевая тема:
Создать новую тему:
Автор:
Действия ...
Форумы / C++ [игнор отключен] [закрыт для гостей] / Ошибка выделения памяти / 7 сообщений из 7, страница 1 из 1
14.12.2006, 19:25
    #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
| Ответить | Цитировать | Написать  
14.12.2006, 19:49
    #34201248
mayton
mayton
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Ошибка выделения памяти
Вот чудик! Хоть бы указал строку , в которой ошибка возникает.

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

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

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

2. Строка
Код: plaintext
memcpy(pasfromreg, buff, strlen(buff));
Пожалуй тут не хватает
Код: plaintext
pasfromreg[strlen(buff)] =  0 ;
раз уж потом ты берёшь его strlen.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
15.12.2006, 13:28
    #34202909
Basilisk
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
| Ответить | Цитировать | Написать  
15.12.2006, 21:29
    #34204513
mayton
mayton
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Ошибка выделения памяти
Любознательный
Если бы я знал в какой строке вываливается ошибка. К тому же она вываливается не всегда.

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

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

Еще лучше - отлаживатся в режиме DEBUG. В этом случае, среда разработки переключит исходники библиотечных классов в параноидально-безопасный режим с различными ASSERTS и т.п.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
Форумы / C++ [игнор отключен] [закрыт для гостей] / Ошибка выделения памяти / 7 сообщений из 7, страница 1 из 1
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение  
Полит. конфиден.  
созд. / загр.: 492ms
Закрыть
start [/forum/topic.php?fid=57&mobile=1&tid=2029811]:
 0ms
get settings:
 8ms
get forum list:
 20ms
check forum access:
 3ms
check topic access:
 3ms
track hit:
 177ms
get topic data:
 12ms
get forum data:
 3ms
get page messages:
 49ms
get tp. blocked users:
 1ms
others: 216ms
total:  492ms
x
x
Закрыть


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