1. PHP / Говнокод #4777

    +172

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    if(!empty($_SESSION["aktion"])){
    //Если сесии не найдено то проверяем если куки
    if(isset($_COOKIE["key"]) && isset($_COOKIE["PHPSESSID"]) && isset($_COOKIE["wrkesh"])){
    //Прогоняем куки через фильтры
    $test["key"] = htmlspecialchars($_COOKIE["key"]);
    $test["key"] = stripslashes($_COOKIE["key"]);
    $test["key"] = mysql_real_escape_string($_COOKIE["key"]);
    $test["PHPSESSID"] = htmlspecialchars($_COOKIE["PHPSESSID"]);
    $test["PHPSESSID"] = stripslashes($_COOKIE["PHPSESSID"]);
    $test["PHPSESSID"] = mysql_real_escape_string($_COOKIE["PHPSESSID"]);
    $test["wrkesh"] = htmlspecialchars($_COOKIE["wrkesh"]);
    $test["wrkesh"] = stripslashes($_COOKIE["wrkesh"]);
    $test["wrkesh"] = mysql_real_escape_string($_COOKIE["wrkesh"]);
    //проверяем если такая запись в бд
    $test_result = mysql_query("SELECT * FROM session WHERE md_5_id='$test[wrkesh]' AND ip='$_SERVER[REMOTE_ADDR]' AND clucc='$test[key]' AND sid='$test[PHPSESSID]'");
    $test_myrow = mysql_fetch_array($test_result);
    if($test_myrow ==true){
    //Если даные с кук и бд совподают то создаём сессию
    $_SESSION["aktive"] ="aktive";
    mysql_close();
    }
    }
    }

    С "Ответов" mail.ru - типа проверка безопасности.

    Запостил: planaric, 28 Ноября 2010

    Комментарии (104) RSS

    • Heil Spellcheck!
      Ответить
      • Fail spellcheck
        Универсальное быдлоправило "Как слышим, так и пишем".
        Ответить
    • Да, у сурьёзных парней - сурьёзная защита.
      Ответить
      • особенно оправдано использование stripslashes
        Ответить
        • она нужна, когда включены магические кавычки.
          Но тут в этом коде, не проверяется включены ли они.
          Ответить
          • ну вот именно. есть вероятность порчи входных данных и возможный секурный фейл.
            а вообще вот пхп за то и раздражает, что от конфиги так сильно зависит работоспособность кода (причем, принцип "не знаешь - не трогай" не срабатывает)
            Ответить
          • Если это реально «ответы мейл.ру», то для таких проектов заранее известны настройки сервера и нет необходимости проверять. Хотя, судя по говнокоду, может тут всё и совсем не так.
            Ответить
            • нет, это - быдло@mail.ru
              Ответить
            • почему думаете, что это сами "ответы@mail.ru"?
              может это кто-то в ответе на вопрос такое написал или вопрос такой сделал.
              ===
              Но даже если известны настройки сервера, всё равно не надо писать без всяких проверок. А вдруг настройки поменяются (например, по ошибке человека) или придётся переехать на другой сервер.
              Ответить
          • показать все, что скрытоvanished
            Ответить
          • показать все, что скрытоwh0cd724173 <a href=http://sildenafilcitrate2017.com/>Sildenafil Without Prescription</a>
            Ответить
    • Я просто в панике, как же я сам до такого не додумался?
      Ответить
    • Автор хотя бы узнал для чего каждая функция нужна.
      Такое ощущение, всё что знал, то и написал
      Ответить
    • Наши руки не для с кук и!
      Ответить
    • $test["key"] = htmlspecialchars($_COOKIE["key"]);
      $test["key"] = stripslashes($_COOKIE["key"]);
      $test["key"] = mysql_real_escape_string($_COOKIE["key"]);

      а никого не смутило, что в $test["key"] попадёт строка, преобразованная только одним способом а не тремя? Или в пэхапэ такой финт прокатит?
      Ответить
      • Именно за это сюда и запостил :)
        Ответить
        • Ну так сам по себе этот факт еще не говнокод, это просто ошибка, баг. Но автор криворук, раз выкладывает ответ с багом.

          А вот говнокод в том, что одна и та же пачка операций выполняется над тремя разными параметрами. Это повод для того, чтобы выделить используемые чистилки в отдельный метод. Или в функцию, не помню как в пхп принято называть. И сразу же видим проявление этой говнистости: баг из-за копипаста размножился 3 раза и исправлять его также надо 3 раза. При чистом подходе исправление в одном методе сразу бы решило проблемы во всех случаях использования.
          Ответить
        • Вдогонку. Но именно для этого случая выделение чистилок в изолированный метод может быть не так нужно, потому что это ответ на мейл-ру и тут наверно важнее, чтобы код был представлен одним куском. Только млин без багов канична ))
          Ответить
        • показать все, что скрытоvanished
          Ответить
          • В застланото с килим преддверие към тях се запъти дребен, наперен като петел за борба мъж, облечен в шит по поръчка копринен костюм с жилетка. Крачките му бяха бързи и енергични като на котарак, който се промъква през открехната врата. Острите му черти винаги извикваха у Пит представата за грифон. Косата му с цвят на тъмен махагон преминаваше в безупречно оформена къса, заострена брада. Властният му поглед пронизваше околните.
            Ответить
      • точно. но похоже mysql_real_escape_string всех спасет
        Ответить
      • показать все, что скрытоvanished
        Ответить
    • Жестоко :-D у нас на работе за такое бы не только руки, но и голову отрубили.
      Ответить
      • у нас в студии за это кочергой по морде (с)
        Ответить
        • неплохой вариант и работать со временем сможет.
          Ответить
    • mail.ru'лит!
      Ответить
    • показать все, что скрытоОткуда у вас исходники Mail.Ru ?
      Я что-то пропустил ?
      (O_o)
      Ответить
      • от хаккиров
        мы им винду, они нам мэйлрушечку
        Ответить
      • Вcё, можете прекратить минусовать.
        Я не сразу врубился, что это ответ а не исходник mail.ru
        ^_^
        Ответить
        • думаю что исходники мыласру ничем не отличаются)
          Ответить
          • жутко неудобная система. они там не родня ЖЖ и одноглазникам?
            Ответить
            • нет, они намного намного намного хуже
              это сборище перловых скриптов 1999го года, запатченного сендмейла и кучки дегенератов)
              Ответить
    • а смысл PHPSESSID хранить в куках здесь?
      Ответить
    • показать все, что скрытоLol
      Ответить
    • показать все, что скрытоwh0cd83825 <a href=http://avodart2016.us.com/>Avodart Sale</a> <a href=http://revia.us.com/>Cheap Revia</a> <a href=http://erektionen.se/levitra.html>levitra vs viagra</a>
      Ответить
    • показать все, что скрытоwh0cd920757 <a href=http://cymbalta2.us/>generic of cymbalta</a> <a href=http://metformin365.us.com/>Buy Metformin</a>
      Ответить
    • показать все, что скрытоwh0cd322786 <a href=http://buyvermox12.gdn/>buy vermox</a> <a href=http://buyzithromax2010.us/>zithromax</a> <a href=http://buyinderal9.us/>buy inderal</a>
      Ответить
    • показать все, что скрытоwh0cd594588 <a href=http://prozac.email/>prozac</a> <a href=http://synthroid.sexy/>synthroid 50</a> <a href=http://cymbalta.photography/>cymbalta</a> <a href=http://abilify.gripe/>aripiprazole abilify</a> <a href=http://buyprozac.casa/>buy prozac online uk</a>
      Ответить
    • показать все, что скрытоwh0cd746369 <a href=http://busparonline.us.com/>Buspar Online</a> <a href=http://tretinoinnorx.us.com/>tretinoin no rx</a> <a href=http://toradol.us.org/>toradol</a>
      Ответить
    • показать все, что скрытоwh0cd583278 <a href=http://albuterol.us.com/>cheap albuterol</a> <a href=http://buysynthroid.us.com/>Generic Synthroid</a>
      Ответить
    • показать все, что скрытоwh0cd909452 <a href=http://erythromycin.desi/>erythromycin</a> <a href=http://sildalis.casa/>sildalis</a> <a href=http://tadalis.desi/>tadalis sx</a> <a href=http://eurax.fund/>eurax</a> <a href=http://fluoxetine.world/>fluoxetine</a>
      Ответить
    • показать все, что скрытоwh0cd800718 <a href=http://levitra2010.top/>levitra</a> <a href=http://buyalbuterol-1.top/>albuterol</a> <a href=http://sildalis0.gdn/>sildalis</a>
      Ответить
    • показать все, что скрытоwh0cd169910 <a href=http://buyarimidex.us.com/>cheapest arimidex</a>
      Ответить
    • показать все, что скрытоwh0cd191425 <a href=http://revianorx.us.com/>revia no rx</a> <a href=http://azithromycin500mgtablets.us.com/>cheapest azithromycin</a>
      Ответить
    • показать все, что скрытоwh0cd169910 <a href=http://glucophage.us.com/>GLUCOPHAGE XR 500</a> <a href=http://buylevitra.us.org/>Generic Levitra</a>
      Ответить
    • показать все, что скрытоwh0cd245785 <a href=http://zithromax.cool/>zithromax</a> <a href=http://costofviagra.business/>viagra</a> <a href=http://methotrexate.shopping/>buy methotrexate online</a> <a href=http://vermox.solutions/>buy vermox</a> <a href=http://benicar.sexy/>benicar 5 mg</a>
      Ответить
    • показать все, что скрытоwh0cd659171 <a href=http://lisinoprilhydrochlorothiazide.us.com/>hydrochlorothiazide online</a>
      Ответить
    • wh0cd485875 <a href=http://viagraprice24.us.org/>Order Viagra</a> <a href=http://buyviagraonline24.us.org/>buy viagra online</a>
      Ответить
    • wh0cd616112 <a href=http://viagra247.us.org/>viaga</a>
      Ответить
    • wh0cd887909 <a href=http://buyeurax-9.gdn/>buy eurax</a> <a href=http://buyanafranil100.top/>anafranil</a> <a href=http://buytadalafil4.top/>generic tadalafil</a> <a href=http://avana2011.top/>super avana</a>
      Ответить
    • wh0cd648955 <a href=http://buyviagra24.us.org/>buy viagra</a> <a href=http://arimidex24.us.org/>arimidex</a> <a href=http://augmentin24.us.org/>buy augmentin</a>
      Ответить
    • wh0cd735033 <a href=http://avodart2017.gdn/>more hints</a> <a href=http://buyazithromycin16.gdn/>azithromycin</a> <a href=http://buyinderal2017.gdn/>buy inderal</a>
      Ответить
    • wh0cd592351 <a href=http://buymetformin6.top/>metformin where to buy</a> <a href=http://buycolchicine100.top/>colchicine</a> <a href=http://tadacip911.gdn/>tadacip</a> <a href=http://crestor2014.gdn/>crestor</a>
      Ответить
    • wh0cd898117 <a href=http://buyneurontin8.top/>neurontin</a> <a href=http://tetracycline17.top/>tetracycline</a> <a href=http://ampicillin2.top/>ampicillin</a>
      Ответить
    • wh0cd39663 <a href=http://erythromycin24.us.org/>erythromycin over counter</a> <a href=http://nolvadex24.us.org/>nolvadex 10</a> <a href=http://genericviagra247.us.org/>online viagra canada</a>
      Ответить
    • wh0cd157468 <a href=http://clomid24.us.org/>Purchase Clomid Online</a> <a href=http://buyviagra24.us.org/>can you buy viagra online</a>
      Ответить
    • wh0cd975121 <a href=http://medrol.us.org/>medrol</a> <a href=http://levofloxacin.us.com/>Levofloxacin</a>
      Ответить
    • wh0cd321663 <a href=http://viagra247.us.org/>viagra price</a> <a href=http://cialisprice24.us.org/>cialis price</a> <a href=http://buyviagra24.us.org/>viagra canadian</a>
      Ответить
    • wh0cd124625 <a href=http://atarax0.top/>atarax</a> <a href=http://phenergan-3.top/>phenergan</a> <a href=http://clindamycin-247.gdn/>generic clindamycin</a> <a href=http://buytadacip2017.gdn/>buy tadacip online</a>
      Ответить
    • wh0cd365829 <a href=http://cephalexin20.top/>cephalexin</a> <a href=http://doxycycline12.top/>doxycycline</a> <a href=http://motilium-2016.gdn/>motilium</a> <a href=http://erythromycin0.top/>erythromycin</a> <a href=http://buycafergot50.top/>cafergot online</a> <a href=http://citalopram365.top/>citalopram</a> <a href=http://albuterol2012.top/>albuterol</a>
      Ответить
    • wh0cd94023 <a href=http://zoloft247.us.org/>Buy Sertraline Online</a> <a href=http://apoteket-sverige.se/>viagra apoteket</a> <a href=http://cialisgeneric2017.com/>cheapest cialis online</a> <a href=http://buycialisonline.us.com/>buy cialis online</a> <a href=http://sumycin.us.com/>sumycin</a>
      Ответить
    • wh0cd593463 <a href=http://viagra-soft.com/>viagra soft tabs</a>
      Ответить
    • wh0cd505149 <a href=http://buytoradol.us.com/>buy toradol</a>
      Ответить
    • wh0cd670478 <a href=http://buyatenolol2015.top/>atenolol metoprolol</a> <a href=http://buycrestor3.gdn/>generic for crestor</a> <a href=http://buylipitor-365.gdn/>lipitor</a>
      Ответить
    • wh0cd713513 <a href=http://cephalexin247.us.org/>keflex 500</a> <a href=http://viagraprice247.us.org/>Cost For Viagra</a>
      Ответить
    • wh0cd942278 <a href=http://buyatarax16.top/>atarax</a> <a href=http://zoloft-0.gdn/>buy zoloft</a> <a href=http://zithromax2014.top/>zithromax</a> <a href=http://anafranil365.gdn/>anafranil</a>
      Ответить
    • wh0cd72503 <a href=http://propecia247.gdn/>where to buy propecia online</a> <a href=http://abilify365.top/>abilify</a> <a href=http://buytrazodone4.top/>trazodone</a>
      Ответить
    • wh0cd659151 <a href=http://amitriptyline24.us.org/>Order Amitriptyline</a> <a href=http://celexa247.us.org/>buy celexa</a>
      Ответить
    • wh0cd72503 <a href=http://abilify365.top/>abilify</a> <a href=http://buyaugmentin8.top/>buy augmentin</a> <a href=http://buyprednisone365.top/>prednisone</a> <a href=http://buycytotec-4.gdn/>cytotec</a> <a href=http://propranolol4.top/>for more info</a>
      Ответить
    • wh0cd124623 <a href=http://cafergot1.gdn/>cafergot online</a> <a href=http://buyerythromycin2016.top/>erythromycin over the counter</a> <a href=http://amoxicillin2.top/>amoxicillin</a>
      Ответить
    • wh0cd202753 <a href=http://costofviagra24.us.org/>viagra without prescription</a> <a href=http://paxil24.us.org/>more</a> <a href=http://zithromax24.us.org/>order zithromax</a>
      Ответить
    • wh0cd583268 <a href=http://prednisone247.us.org/>prednisone</a> <a href=http://lisinopril24.us.org/>lisinopril</a> <a href=http://prozac24.us.org/>prozac</a>
      Ответить
    • wh0cd659150 <a href=http://augmentin247.us.org/>Augmentin</a> <a href=http://wellbutrin24h.us.org/>wellbutrin drug</a>
      Ответить
    • wh0cd191425 <a href=http://revianorx.us.com/>revia no rx</a> <a href=http://cephalexin500mg.us.org/>cephalexin 500mg</a> <a href=http://amoxicillin2016.us.com/>cheapest amoxicillin</a> <a href=http://levaquin.us.com/>GENERIC LEVAQUIN</a> <a href=http://cheapcialisonline.us.com/>cialis</a>
      Ответить
    • wh0cd571957 <a href=http://cialisgeneric24.us.org/>Cialis Generic</a> <a href=http://provera24h.us.org/>generic provera</a> <a href=http://allopurinol24.us.org/>allopurinol no prescription</a>
      Ответить
    • wh0cd766751 <a href=http://viagraonline24h7d.us.org/>additional info</a>
      Ответить
    • wh0cd103105 <a href=http://prednisone247.us.org/>prednisone</a>
      Ответить
    • wh0cd320555 <a href=http://nolvadex365.us.com/>cheap nolvadex</a> <a href=http://discountviagra.us.com/>viagra pills</a> <a href=http://citalopram2016.us/>citalopram</a> <a href=http://paxilnorx.us.com/>generic paxil cost</a> <a href=http://singulairgeneric.us.com/>singulair 4 mg</a>
      Ответить
    • wh0cd224274 <a href=http://prozac24.us.org/>prozac price</a> <a href=http://tamoxifen247.us.org/>Nolvadex</a> <a href=http://viagraonline24h7d.us.org/>Lady Viagra</a>
      Ответить
    • wh0cd278634 <a href=http://buynolvadex10.top/>nolvadex</a> <a href=http://buymotilium6.gdn/>motilium</a> <a href=http://buyrobaxin12.top/>methocarbamol robaxin</a> <a href=http://indocin11.top/>indocin</a> <a href=http://buycymbalta2015.gdn/>cymbalta</a> <a href=http://avana10.gdn/>avana</a> <a href=http://buycolchicine-4.gdn/>buy colchicine</a>
      Ответить
    • wh0cd527785 <a href=http://advairdiskus24.us.org/>advair diskus</a> <a href=http://buyviagraonline247.us.org/>Buy Viagra Online</a> <a href=http://genericcialis247.us.org/>Cialis Prices</a>
      Ответить
    • wh0cd190311 <a href=http://buyrimonabant10.gdn/>buy rimonabant</a> <a href=http://avodart-1.top/>avodart</a> <a href=http://avodart0.top/>avodart</a> <a href=http://buycolchicine100.top/>colchicine gout</a> <a href=http://buyavodart16.top/>buy avodart</a> <a href=http://buycephalexin2017.top/>buy cephalexin</a> <a href=http://buyviagrasoft10.top/>viagra soft</a>
      Ответить
    • wh0cd321675 <a href=http://buyrevia10.top/>revia</a> <a href=http://seroquel17.top/>seroquel</a> <a href=http://buyelimite-2016.gdn/>buy elimite</a> <a href=http://propranolol100.top/>propranolol buy</a>
      Ответить
    • wh0cd158583 <a href=http://buystrattera11.top/>strattera</a> <a href=http://cialis2014.gdn/>cefixime</a> <a href=http://celexa1.top/>celexa</a> <a href=http://anafranil365.gdn/>buy anafranil</a> <a href=http://retina15.gdn/>retin a</a> <a href=http://elocon3.gdn/>read full article</a> <a href=http://buyrevia7.top/>revia</a> <a href=http://avodart-1.top/>avodart</a>
      Ответить
    • wh0cd376037 <a href=http://anafranil.us.com/>buy anafranil</a> <a href=http://buyviagraonline2017.com/>where can buy viagra</a> <a href=http://vasotec.us.com/>VASOTEC</a>
      Ответить
    • wh0cd311467 <a href=http://celexa.us.com/>Celexa</a>
      Ответить
    • wh0cd431511 <a href=http://neurontin11.top/>neurontin online</a> <a href=http://buyfurosemide250.gdn/>where to buy furosemide</a> <a href=http://buylipitor3.gdn/>buy lipitor</a> <a href=http://buytriamterene0.gdn/>triamterene weight loss</a> <a href=http://buyrobaxin-9.gdn/>robaxin muscle relaxant</a> <a href=http://buypropranolol-6.top/>propranolol</a>
      Ответить
    • wh0cd853945 <a href=http://synthroidnorx.us.com/>citation</a> <a href=http://lisinopril24.us.org/>Buy Lisinopril</a> <a href=http://suhagra24.us.org/>suhagra</a> <a href=http://propecia3.us/>where to buy finasteride online</a>
      Ответить
    • wh0cd485871 <a href=http://tetracycline2016.us.com/>CHEAP TETRACYCLINE</a> <a href=http://priceofcialis.us.com/>cialis</a> <a href=http://atenolol24.us.org/>atenolol lowest cost</a> <a href=http://lipitor247.us.com/>Lipitor</a>
      Ответить
    • wh0cd540236 <a href=http://fluoxetine24.us.org/>order fluoxetine</a>
      Ответить
    • Растёр тебя с кишками крысы, сдохшей в страстную пятницу, добавил мышьяка и свинцовый глет.
      Рекомендовал как яд для смазывания стрел.
      Ответить
    • показать все, что скрытоvanished
      Ответить
    • показать все, что скрытоvanished
      Ответить
    • показать все, что скрытоvanished
      Ответить
    • показать все, что скрытоvanished
      Ответить
    • показать все, что скрытоvanished
      Ответить

    Добавить комментарий