Надо срочно потестить маленькие скрипты

Discussion in 'PHP' started by Дикс, 3 Mar 2008.

  1. Дикс

    Дикс Elder - Старейшина

    Joined:
    16 Apr 2006
    Messages:
    1,194
    Likes Received:
    227
    Reputations:
    26
    Здрасьте,
    сделал вот тестовое задание для устройства на работу пхп-программером, сами понимаете - всё должно быть идеально.

    От вас просьба потестить его во всем - начиная от валидности и актуальности html, до уязвимости и быстродействия скриптов.

    http://dev.centerft.com/blog/admin/index.php

    sources (~9 kb):
    http://dev.centerft.com/blog/admin/admin_utility.zip

    Естественно не за спасибо. Тем кто активно поможет, помогу яндекс деньгами :)
     
  2. spider-intruder

    spider-intruder Elder - Старейшина

    Joined:
    9 Dec 2005
    Messages:
    700
    Likes Received:
    339
    Reputations:
    37
    Xss в имени пользователя :)
     
    1 person likes this.
  3. GreenTiger

    GreenTiger Banned

    Joined:
    7 Dec 2007
    Messages:
    62
    Likes Received:
    10
    Reputations:
    0
    При добавлении нового:
    Некоторые поля уязвимы..
    '"/><script>alert(/1/)</script>

    Активка
     
  4. imajo.ati

    imajo.ati Banned

    Joined:
    21 Feb 2008
    Messages:
    232
    Likes Received:
    62
    Reputations:
    8
    ничего не фильтруется кроме login =)

    =) при редактировании абсолютно ничего не фильтруется
     
    #4 imajo.ati, 3 Mar 2008
    Last edited: 3 Mar 2008
    1 person likes this.
  5. spider-intruder

    spider-intruder Elder - Старейшина

    Joined:
    9 Dec 2005
    Messages:
    700
    Likes Received:
    339
    Reputations:
    37
    размеры полей контролируй
     
  6. spider-intruder

    spider-intruder Elder - Старейшина

    Joined:
    9 Dec 2005
    Messages:
    700
    Likes Received:
    339
    Reputations:
    37
    Год рождения можно поставить Нулевым так не бывает :)
     
  7. bul.666

    bul.666 булка

    Joined:
    6 Jun 2006
    Messages:
    719
    Likes Received:
    425
    Reputations:
    140
    Проверь на валидность редактирование пользователей
     
  8. spider-intruder

    spider-intruder Elder - Старейшина

    Joined:
    9 Dec 2005
    Messages:
    700
    Likes Received:
    339
    Reputations:
    37
    ОНОТОЛЕЙЙЙЙ - Xss почини наконец )))
     
  9. diehard

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

    Joined:
    30 Sep 2007
    Messages:
    442
    Likes Received:
    266
    Reputations:
    15
    Не связаны логически между собой поля Age (Возраст) и Birthday (Дата рождения).
     
  10. spider-intruder

    spider-intruder Elder - Старейшина

    Joined:
    9 Dec 2005
    Messages:
    700
    Likes Received:
    339
    Reputations:
    37
    А как вам это :)
     
  11. spider-intruder

    spider-intruder Elder - Старейшина

    Joined:
    9 Dec 2005
    Messages:
    700
    Likes Received:
    339
    Reputations:
    37
    А это )))
     
  12. spider-intruder

    spider-intruder Elder - Старейшина

    Joined:
    9 Dec 2005
    Messages:
    700
    Likes Received:
    339
    Reputations:
    37
    Кончайте играться :)
    Пусть ОноТолеЙ думает что ему с этим делать. Толи забить на работу програмистом ПХП толи исправлять баги. ОнотолЕй как будешь готов говори - мы снова разнесем твой блог )))
     
  13. imajo.ati

    imajo.ati Banned

    Joined:
    21 Feb 2008
    Messages:
    232
    Likes Received:
    62
    Reputations:
    8
    если можно писать русские буквы в пасс, то кодируй их в url, пересылай, а потом раскодируй и пиши в бд. иначе ни у одного юзера не подойдёт пасс к его логину
     
  14. diehard

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

    Joined:
    30 Sep 2007
    Messages:
    442
    Likes Received:
    266
    Reputations:
    15
    Осторожно: поифреймлено :)))
     
  15. Дикс

    Дикс Elder - Старейшина

    Joined:
    16 Apr 2006
    Messages:
    1,194
    Likes Received:
    227
    Reputations:
    26
    >>Год рождения можно поставить Нулевым так не бывает :)
    эта пох, на ачате ваще можно 31 февраля в качестве даты рождения указать.

    >>размеры полей контролируй
    ёпт ) так увлёкся интерфейсом, что забыл такие простые весчи.

    >>Не связаны логически между собой поля Age (Возраст) и Birthday (Дата рождения).
    я об этом думал, но в ТЗ написано что бд должна содержать поля "возраст" и "дата рождения". Они уже не вяжутся. так что оставим.
     
  16. diehard

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

    Joined:
    30 Sep 2007
    Messages:
    442
    Likes Received:
    266
    Reputations:
    15
    Дикс

    так значит под датой рождения подразумевается день+месяц, без года
     
  17. imajo.ati

    imajo.ati Banned

    Joined:
    21 Feb 2008
    Messages:
    232
    Likes Received:
    62
    Reputations:
    8
    ещё один нюанс. при отключенном яваскрипте пропадает поле для ввода месяца рождения
     
  18. spider-intruder

    spider-intruder Elder - Старейшина

    Joined:
    9 Dec 2005
    Messages:
    700
    Likes Received:
    339
    Reputations:
    37
    ну вроде кое что пофиксил - тестируйте!
     
  19. imajo.ati

    imajo.ati Banned

    Joined:
    21 Feb 2008
    Messages:
    232
    Likes Received:
    62
    Reputations:
    8
    при надписи что все поля обязательны, можно зарегиться указав только логин и пароль
     
  20. Дикс

    Дикс Elder - Старейшина

    Joined:
    16 Apr 2006
    Messages:
    1,194
    Likes Received:
    227
    Reputations:
    26
    блин, точно! надо год убирать.

    я перегрелся чуток =\

    [​IMG]