mysql_real_escape_string / Говнокод #5099 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
$sql = "DELETE  FROM ".PREFIX."_orders
            WHERE id = '".$order_id."'
	    ORDER BY order_date DESC";

Запостил: 1_and_0 1_and_0, (Updated )

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

    • ava wmmorgun # Ссылка на оригинал 0 ⇈

      Возможно, а возможно там дальше еще делается $sql .= " LIMIT 1";
      Хотя, в любом случае, хорошим такой код не назовёшь.

      P.S. чтобы не было вопросов: тут не указано, что id - это уникальный индекс
      Ответить

      • ava bugmenot # Ссылка на оригинал 0 ⇈

        это PHP
        а значит MySQL
        следовательно автору молотком вбили про синтетический ключ id

        DELETE ... ORDER BY i`lol`d
        Ответить

        • ava Lure Of Chaos # Ссылка на оригинал 0 ⇈

          > это PHP

          следовательно[color=red],[/red] автор сего вообще не дал труда себе разобраться в чем либо. Так, скопировал код с форума, подправил. Залил на продакшн, убедился, что не кидает fatal error, и доволен
          Ответить

        • ava bugmenot # Ссылка на оригинал 0 ⇈

          а, дада, еще следы молотка на "обязательно первой колонкой"
          Ответить

        • ava wmmorgun # Ссылка на оригинал 0 ⇈

          В любом случае оторвано от контекста это не настолько говнисто, как могло бы быть.
          Может включен register_globals, и это например колбек на удаление по аджаксу, с кодом:
          <?php 
include 'config.inc';
mysql_connect($dbadres, $dbluzer, $dbporol) or die("Мне урезали зарплату! Вася.");
mysql_selectdb($dbimya) or die("Куда дели базу?");
$sql = "DELETE  FROM ".PREFIX."_orders
            WHERE id = '".$order_id."'
	    ORDER BY order_date DESC";
mysql_query($sql) or die('Умри хакер');
echo '{"success": true}';
?>


          Реквестирую кусочек побольше
          Ответить

          • ava Lure Of Chaos # Ссылка на оригинал 0 ⇈

            вот за такой аякс нужно точно урезать зарплату. Бо удаляй не хочу кто хошь и что хошь
            Ответить

            • ava bugmenot # Ссылка на оригинал 0 ⇈

              sucksass: true
              поубивал бы за "ajax"
              а за JSON где не надо в сортир
              Ответить

            • ava wmmorgun # Ссылка на оригинал 0 ⇈

              Я просто показал, что оторванный от контекста код может не показывать степень своей говнистости)
              За такой код надо как минимум увольнять )
              Ответить

  • ava guest # Ссылка на оригинал 0

    Я админ этого сайта. Я вас всех ща забаню


    заведу и отибу.
    Ответить

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

Помни, guest, за тобой могут следить!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓