Member 13512434 Ответов: 2

Oledbexception был необработан - C# требуется объяснение, пожалуйста

Я создаю логин пользователя для своего программного обеспечения, которое подключено к Access, я продолжаю получать ту же ошибку, которая выделяет строку

'OleDbDataReader reader = команда.Метода executereader();' высказывание...

'Необработанное исключение типа 'System.Data.OleDb.OleDbException' произошло в System.Data.dll

Дополнительная информация: синтаксическая ошибка (отсутствует оператор) в выражении запроса 'Username= 'MGRjs' и пароль 'Candy')'.'


private void LoginButton_Click(object sender, EventArgs e)
{
        connection.Open();
        OleDbCommand command = new OleDbCommand();
        command.Connection = connection;
        command.CommandText = "select * from NewUser where Username= '" + IDtbx.Text + "' and Password'" + PSWtbx.Text + "')";

        OleDbDataReader reader = command.ExecuteReader();
        int count = 0;
        while (reader.Read())
        {
            count = count + 1;
        }

        if (count == 1)
        {

            MainSystem mainForm = new MainSystem();
            mainForm.FormClosed += new FormClosedEventHandler(Login_FormClosed);
            mainForm.Show();
            this.Hide();
            LoginError.Visible = false;
            connection.Close();
            connection.Dispose();
        }

        else
        {
            LoginError.Visible = true;
        }
        connection.Close();


Есть предложения?

Что я уже пробовал:

Повторное подключение базы данных access, подключение другой базы данных

2 Ответов

Рейтинг:
13

OriginalGriff

Две вещи, обе важные:
1) Никогда не объединяйте строки для построения SQL-команды. Это оставляет вас широко открытыми для случайной или преднамеренной атаки SQL-инъекции, которая может уничтожить всю вашу базу данных. Вместо этого используйте параметризованные запросы.

Когда вы объединяете строки, вы вызываете проблемы, потому что SQL получает такие команды, как:

SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
Цитата, добавленная пользователем, завершает строку в том, что касается SQL, и вы получаете проблемы. Но могло быть и хуже. Если я приду и наберу вместо этого: "x';DROP TABLE MyTable;--", то SQL получит совсем другую команду:
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Которые SQL видит как три отдельные команды:
SELECT * FROM MyTable WHERE StreetAddress = 'x';
Совершенно правильный выбор
DROP TABLE MyTable;
Вполне допустимая команда "удалить таблицу" 
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?

2) Никогда не храните пароли в открытом тексте - это серьезная угроза безопасности. Здесь есть некоторая информация о том, как это сделать: Хранение паролей: как это сделать.[^]

Исправьте их, и ваша ошибка будет очевидна: между "паролем" и цитатой вокруг введенного пользователем текста было бы"=".

Рейтинг:
0

F-ES Sitecore

Изучите справку sql в вашей команде

command.CommandText = "select * from NewUser where Username= '" + IDtbx.Text + "' and Password'" + PSWtbx.Text + "')";


Похоже ли это на действительный SQL? 

and Password'Candy'


допустимый SQL?

Измените его на

command.CommandText = "select * from NewUser where Username= '" + IDtbx.Text + "' and Password='" + PSWtbx.Text + "')";


Однако вам действительно нужно отказаться от этого способа выполнения SQL и вместо этого использовать параметризованные запросы.