[Вопрос] Неправильная работа strcmp

Неправильная работа strcmp

Здравствуйте, после регистрации у меня идет авторизация игрока. Чтобы сверка паролей была. Проблема заключается в том, что пароль принимает любой. Помогите пожалуйста.

PHP код:

if(dialogid == 0) { if(!response) return Kick(playerid); else if(response) { new string[280]; format(string,sizeof(string),"{FFDEAD}Здравствуй, %s \n\n\a Ты зашел на сервер AscarY(0.3.7)[RUS/UA]\n\a Игровой Режим: AscarY: LS/SF/LV\n\a На сервере: дома(ов)\n\a Бизнесов: \n\a Сбизнесов: \n\a Сайт Сервера: AscarY-Rpg.Ru\n\nИспользуемый тобой ник свободен.\nЗарегистрируйся.\n\nЗадай пароль от аккаунта:",sendername); if(!strlen(inputtext)) return ShowPlayerDialog(playerid, 0, DIALOG_STYLE_PASSWORD, "Регистрация",string,"Далее","Отмена"); new account[37]; format(account,sizeof(account),"users/%s.ini",pNick(playerid)); format(string,sizeof(string),"{FFDEAD}Здравствуй, %s \n\n\a Ты зашел на сервер AscarY (0.3.7)[RUS/UA]\n\a Игровой режим: AscarY: LS/SF/LV(RPG)\n\a На сервере: дома(ов)\n\a Бизнесов: \n\a Сбизнесов: \n\a Сайт Сервера: AscarY-Rpg.Ru\n\nЭтот аккаунт уже зарегистрирован.\n\nВведи пароль от аккаунта:",sendername); if(fexist(account)) return ShowPlayerDialog(playerid, 1, DIALOG_STYLE_PASSWORD,"Авторизация",string,"Вход","Отмена"); else { ShowPlayerDialog(playerid, 2, DIALOG_STYLE_PASSWORD,"Авторизация","{FFDEAD}Аккаунт зарегистрирован\nВведи пароль для продолжения:","Вход","Отмена"); CreateAccount(playerid, inputtext); return 1; } } } else if(dialogid == 2) { if(!response) return Kick(playerid); else if(response) { if(!strlen(inputtext)) { ShowPlayerDialog(playerid, 2, DIALOG_STYLE_PASSWORD,"Авторизация","{FFDEAD}Аккаунт зарегистрирован\nВведи пароль для продолжения:","Вход","Отмена"); SendClientMessage(playerid, COLOR_WHITE, "Пароль не подошел! Попробуй еще раз."); return 1; } else { new password[64], account[64]; new iniFile = ini_openFile(account); ini_getString(iniFile, "Пароль",password); ini_closeFile(iniFile); if(!strcmp(inputtext, password, false)) return OnPlayerLogin(playerid); else if(strcmp(inputtext, password, false)) { ShowPlayerDialog(playerid, 2, DIALOG_STYLE_PASSWORD,"Авторизация","{FFDEAD}Аккаунт зарегистрирован\nВведи пароль для продолжения:","Вход","Отмена"); SendClientMessage(playerid, COLOR_WHITE, "Пароль не подошел! Попробуй еще раз."); return 1; } } } }

Если в strcmp одна из строк пустая, функция вернёт 0, как будто обе строки одинаковы, в независимости от содержимого другой строки. Очень старый баг, который присутствует чуть ли не с самого начала существования Pawn.
Обойти его можно, добавив дополнительную проверку на нулевую строку.

PHP код:

if((inputtext[0] == EOS) || (strcmp(inputtext, password, false) != 0))

P.S.: Я не стал добавлять проверку на пустоту строки password, т.к. она должна проверяться ещё при регистрации/смене пароля.

P.P.S.: Сменил название темы, чтобы другим было проще найти её. Не забудьте отписаться, если проблема решена, перемещу тему в архив.

Я наверное недопонял, куда эту проверку добавить нужно?

После ini_CloseFile.

PHP код:

ini_closeFile(iniFile); if((inputtext[0] == EOS) || (strcmp(inputtext, password, false) != 0)) { ShowPlayerDialog(playerid, 2, DIALOG_STYLE_PASSWORD,"Авторизация","{FFDEAD}Аккаунт зарегистрирован\nВведи пароль для продолжения:","Вход","Отмена"); return SendClientMessage(playerid, COLOR_WHITE, "Пароль не подошел! Попробуй еще раз."); } return OnPlayerLogin(playerid);

К сожалению, так же заходит с любым паролем.
Как поменял диалог:

PHP код:

else if(dialogid == 2) { if(!response) return Kick(playerid); else if(response) { if(!strlen(inputtext)) { ShowPlayerDialog(playerid, 2, DIALOG_STYLE_PASSWORD,"Авторизация","{FFDEAD}Аккаунт зарегистрирован\nВведи пароль для продолжения:","Вход","Отмена"); SendClientMessage(playerid, COLOR_WHITE, "Пароль не подошел! Попробуй еще раз."); return 1; } else { new password[64], account[64]; new iniFile = ini_openFile(account); ini_getString(iniFile, "Пароль",password); ini_closeFile(iniFile); if((inputtext[0] == EOS) || (strcmp(inputtext, password, false) != 0)) { ShowPlayerDialog(playerid, 2, DIALOG_STYLE_PASSWORD,"Авторизация","{FFDEAD}Аккаунт зарегистрирован\nВведи пароль для продолжения:","Вход","Отмена"); return SendClientMessage(playerid, COLOR_WHITE, "Пароль не подошел! Попробуй еще раз."); } return OnPlayerLogin(playerid); } } }

Вопрос к сожалению, так и не решен(

Есть такая вероятность что, строка "Пароль" в файле игрока users/%s.ini ничего не содержит, в связи с чем в массив password[64] ничего не записывается, strcmp сравнивает введенный пароль inputtext и пустой массив password[64], и возвращает 0.
Проверьте в файле игрока users/%s.ini строку "Пароль", и ищете причину почему пароль не записывается.

Да кто говорит что не записывается? Все записывается, проблема в другом, не знаю в чем.

Цитата:

Сообщение от Skyline

Да кто говорит что не записывается? Все записывается, проблема в другом, не знаю в чем.

Выведи логи, но у меня вопрос где путь к аккаунту?

PHP код:

else if(dialogid == 2) { if(!response) return Kick(playerid); else if(response) { if(!strlen(inputtext)) { ShowPlayerDialog(playerid, 2, DIALOG_STYLE_PASSWORD,"Авторизация","{FFDEAD}Аккаунт зарегистрирован\nВведи пароль для продолжения:","Вход","Отмена"); SendClientMessage(playerid, COLOR_WHITE, "Пароль не подошел! Попробуй еще раз."); return 1; } else { new password[64], account[64]; new iniFile = ini_openFile(account); ini_getString(iniFile, "Пароль",password); printf("Диалог авторизации | Пароль: %s | Вводимый пароль: %s", password, inputtext); ini_closeFile(iniFile); if((inputtext[0] == EOS) || (strcmp(inputtext, password, false) != 0)) { ShowPlayerDialog(playerid, 2, DIALOG_STYLE_PASSWORD,"Авторизация","{FFDEAD}Аккаунт зарегистрирован\nВведи пароль для продолжения:","Вход","Отмена"); return SendClientMessage(playerid, COLOR_WHITE, "Пароль не подошел! Попробуй еще раз."); } return OnPlayerLogin(playerid); } } }

Походу мой косяк, поспешил с выводами и только зря переименовал тему. Но доля правды в том выводе была.

Таки да, строковая переменная account не инициализирована (пропустили format?), из-за чего mxINI не открывает файл.
Функция ini_getString возвращает из несуществующего файла пустую строку, после чего имеет место баг strcmp, описанный мной выше. Таким образом, к аккаунту подходят все пароли.

Цитата:

Сообщение от Daniel_Cortez

Походу мой косяк, поспешил с выводами и только зря переименовал тему. Но доля правды в том выводе была.

Таки да, строковая переменная account не инициализирована (пропустили format?), из-за чего mxINI не открывает файл.
Функция ini_getString возвращает из несуществующего файла пустую строку, после чего имеет место баг strcmp, описанный мной выше. Таким образом, к аккаунту подходят все пароли.

Да, проблема оказалась в этом. Не вывел format, по этому принимал любой пароль, из-за того что ник был неизвестен.
Спасибо большое)