Решения на основе HTTP или SQL Server для внедрения SQL - PullRequest
1 голос
/ 09 декабря 2008

Я работаю с SQL Server 2005 и Windows Server 2000 и задаюсь вопросом, существуют ли какие-либо «автоматические» способы блокирования атак SQL-инъекций, пока я укрепляю свой код.

Некоторые предположили, что есть способы:

  1. Вставьте какой-нибудь модуль ISAPI или HTTP, который фильтрует пост-запрос и строки запроса для символов, ориентированных на инъекцию, и не выполняет запрос, прежде чем он попадет в приложение. Большинство из этих решений ориентированы на IIS 6 или выше. Я бегу 5.
  2. Гарантируйте, что каждый объект команды запускает только одну команду SQL за один раз.

Есть другие идеи для моей конфигурации?

Ответы [ 5 ]

1 голос
/ 07 марта 2009

Всегда очищать пользовательский ввод

  1. если бы вы запретили ', вы бы немедленно пошли как-нибудь сделать свой код более безопасным
  2. если ваш запрос ожидает целое число, убедитесь, что входные данные являются целыми числами. и т.д.
1 голос
/ 10 декабря 2008

Когда у меня было несколько попыток атак с помощью инъекций на моем сервере, я беспокоился, что они занимают ненужные ресурсы. Я написал (взломан!) HttpModule на c #, который отфильтровывал бы большинство xss и sql инъекционных атак. Код вставлен ниже вместе с разделом конфигурации, необходимым для использования веб-сайта. Он должен быть помещен в проект и скомпилирован в файл WebSecurityFilter.dll, на который должен ссылаться веб-проект (или иным образом удаленный в каталог bin).

Это будет работать только с asp.net, так что, надеюсь, ваш сайт основан на asp.net (я спрашивал в комментарии, но не получил ответа).

Раздел веб-конфигурации (добавьте его в раздел :

  <add name="SecurityHttpModule" type="WebSecurityFilter.SecurityHttpModule, WebSecurityFilter" />

Код для модуля (SecurityHttpModule.cs):

using System;
using System.Collections.Generic;
using System.Text;
using System.Web;
using System.Text.RegularExpressions;

namespace WebSecurityFilter
{
    class SecurityHttpModule : IHttpModule
    {
        class RegexWithDesc : Regex
        {
            string _errorText;

            public string ErrorText
            {
                get { return _errorText; }
            }

            public RegexWithDesc(string regex, RegexOptions options, string errorText)
                :base(regex, options)
            {
                _errorText = errorText;
            }
        }
        /// <summary>
        /// error text displayed when security violation is detected
        /// </summary>
        private string _errorhtml =
        @"<!DOCTYPE html PUBLIC ""-//W3C//DTD XHTML 1.1//EN"" ""http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd"">" +
        @"<html xmlns=""http://www.w3.org/1999/xhtml"" >" +
        @"<body style=""background:black;"">" +
        @"<table style=""width:100%"" >" +
        @"<tr><td align=""center"">" +
        @"<div style=""border:3px solid red;text-align:center;width:95%;color:red;padding:10px;text-decoration:blink;"">" +
        @"SECURITY VIOLATION" +
        @"<br/>" +
        //@"<br/>" +
        //@"go away" +
        //@"<br/>" +
        @"<br/>" +
        @"{0}" +
        @"<br/>" +
        @"</div>" +
        @"</td></tr>" +
        @"</table>" +
        @"</body>" +
        @"</html>";

        // regex for default checks
        // http://www.securityfocus.com/infocus/1768
        static RegexOptions _defaultRegexOptions = RegexOptions.Compiled | RegexOptions.IgnoreCase | RegexOptions.IgnorePatternWhitespace;

        RegexWithDesc[] _regexCollection = new RegexWithDesc[] 
        { 
            new RegexWithDesc(@"((¼|<)[^\n]+(>|¾)*)|javascript|unescape", _defaultRegexOptions, "XSS 1"), //3.3
            // new RegexWithDesc(@"(\')|(\-\-)", _defaultRegexOptions, "SQL 1"), //2.1
            new RegexWithDesc(@"(=)[^\n]*(\'|(\-\-)|(;))", _defaultRegexOptions, "SQL 2"),    //2.2
            //new RegexWithDesc(@"\w*(\')(or)", _defaultRegexOptions, "SQL 3"),  //2.3
            new RegexWithDesc(@"(\')\s*(or|union|insert|delete|drop|update|create|(declare\s+@\w+))", _defaultRegexOptions, "SQL 4"),   //2.4
            new RegexWithDesc(@"exec(((\s|\+)+(s|x)p\w+)|(\s@))", _defaultRegexOptions, "SQL 5")    //2.5
        };
        #region IHttpModule Members

        public void Dispose()
        {
           // nothing to do
        }

        public void Init(HttpApplication context)
        {
            context.BeginRequest += new EventHandler(context_BeginRequest);
        }

        void context_BeginRequest(object sender, EventArgs e)
        {
            try
            {
                List<string> toCheck = new List<string>();
                foreach (string key in HttpContext.Current.ApplicationInstance.Request.QueryString.AllKeys)
                {
                    toCheck.Add(HttpContext.Current.ApplicationInstance.Request[key]);
                }
                foreach (string key in HttpContext.Current.ApplicationInstance.Request.Form.AllKeys)
                {
                    toCheck.Add(HttpContext.Current.ApplicationInstance.Request.Form[key]);
                }
                foreach (RegexWithDesc regex in _regexCollection)
                {
                    foreach (string param in toCheck)
                    {
                        string dp = HttpUtility.UrlDecode(param);
                        if (regex.IsMatch(dp))
                        {
                            HttpContext.Current.ApplicationInstance.Response.Write(string.Format(_errorhtml, regex.ErrorText));
                            HttpContext.Current.ApplicationInstance.CompleteRequest();
                            return;
                        }
                    }
                }

            }
            catch (System.Threading.ThreadAbortException x)
            {
                throw;
            }
            catch (Exception ex)
            {
                HttpContext.Current.ApplicationInstance.Response.Write(string.Format(_errorhtml, "Attack Vector Detected"));
                HttpContext.Current.ApplicationInstance.Response.Write(string.Format(_errorhtml, ex.GetType().ToString()));
                HttpContext.Current.ApplicationInstance.CompleteRequest();
                return;
            }
        }

        #endregion
    }
}

Надеюсь, все отформатировано нормально ...

Сегодня вечером я попытаюсь опубликовать ссылку на весь проект в формате zip.

1 голос
/ 09 декабря 2008

Это предлагаемое решение:

Гарантия того, что каждый объект команды выполняет только одну команду SQL на время.

на самом деле не предотвращает инъекцию. Например, злоумышленник может ввести запрос на вход в систему для входа без учетных данных. Рассмотрим:

"SELECT COUNT(*) FROM Users WHERE UserId = '{0}' AND PassHash = '{1}'"

Этот шаблон может быть введен с идентификатором пользователя:

' OR 1=1 --

Уступая:

"SELECT COUNT(*) FROM Users WHERE UserId = '' OR 1=1 --' AND PassHash = 'sdfklahsdlg'"

Сосредоточьте свои усилия на устранении уязвимости из кода.

1 голос
/ 10 декабря 2008

Убедитесь, что все вызовы вашей базы данных используют хранимые процедуры или параметризованные запросы.

1 голос
/ 09 декабря 2008

В целом не существует автоматического решения для защиты от внедрения SQL. SQL-инъекция - это ошибка приложения, а не ошибки базы данных.

Решение заключается в том, чтобы вы делали обзор кода во всех случаях, когда вы выполняете SQL, который интерполирует данные приложения в запрос.

...