Уязвимость Classic ASP Cookie

Question

У меня есть простая страница входа на классическую страницу asp.

Он принимает поле имени пользователя из формы («ООН») и пароль («PW») и выполняет следующие действия с ними:

<% 
    un = newstr(request.form("un")) 
    pw = newstr(request.form("pw")) 

    SQL = "SELECT * from my_table WHERE un = '"&un&"' AND pw = '"&pw&"'" 
    set cRS = oConn.Execute(SQL) 

    if cRS.EOF then 
    %> 

     <p>Unable to log you in. Please <a href="default.asp">try logging in again</a>.</p> 

    <% 
    elseif NOT cRS.EOF then 

     Response.Cookies("test") = "jeQmV4'QG)Eu'N-XSTC;pZeuwqUsjBdVv>Sqr!]ZhzB{dJ'p-#cYSdwY" 
     Response.Cookies("test").Expires = Date() + 365 

     response.redirect "main.asp" 

    end if 

end if 
%> 

Тогда у меня есть некоторые простые проверки в верхней части каждого страница которого содержу я хочу только зарегистрированных пользователей, чтобы иметь возможность видеть, что делает это:

<% 
test = Request.Cookies("test") 
if test = "" OR test <> "jeQmV4'QG)Eu'N-XSTC;pZeuwqUsjBdVv>Sqr!]ZhzB{dJ'p-#cYSdwY" then response.redirect("default.asp") 
%> 

Я хотел проверить, - это то, что по наивности просто?

Может кто-то легко взломать мой сайт, например. установив файл cookie на свой компьютер под названием «test» и значение которого = «jeQmV4'QG) Eu'N-XSTC; pZeuwqUsjBdVv> Sqr!] ZhzB {dJ'p- # cYSdwY"?

Или не было бы маловероятным, чтобы кто-то догадался, что строковое значение «jeQmV4'QG» Eu'N-XSTC; pZeuwqUsjBdVv> Sqr!] ЖЖБ {dJ'p- # cYSdwY "?

Благодаря

Answer 1

Во-первых, кто-то может повредить вашу систему, пытаясь передать имена пользователей как

«; DROP TABLE Пользователи

http://xkcd.com/327/

Всегда, всегда используйте параметризованные запросы.

Не доверяйте пользовательскому интерфейсу за все, что связано с безопасностью. Клиент может наблюдать все файлы cookie, которые вы установили, и возвращать любые файлы cookie обратно к вам, к которому он хочет. Если это не HTTPS, кто-то посередине может также наблюдать, как cookie отправляется туда и обратно.

Не заново изобретайте рамки безопасности. Их очень сложно добиться.