PHP — Говнокод.ру

PHP / Говнокод #2938

+144

1

// кода не выкладываю, по соображениям безопасности

Не писаным правилом безопасности (при создание сайтов) считается, что доступ к админке в целом и конкретно к отдельным её частям должен иметь только авторизированный пользователь. На самом деле все обстоит не так, с уже известной вам горе компании (КСК), для примера привожу безобидное (дабы не пошли ломать ресурсы) пренебрежением безопасности: http://amisharin.ru/admin/ieeditor/ieeditor.php?vname=htmlcode
Как видно не авторизированный пользователь получает доступ к редактору (который, кстати, стабильно работает, только в IE6).

MoLe-X, 06 Апреля 2010

Комментарии (13)

PHP / Говнокод #2935

+171

01
02
03
04
05
06
07
08
09
10
11
12

// Сначала я увидел такой самонадеянный способ "послать нахуй все инъекции" (magic_quotes_gpc при этом вообще не проверялись :-))

# Посылаем смело нахуй все инъекции
foreach($_GET as $name => $value){
    $_GET[$name] = mysql_real_escape_string($value);
}
foreach($_POST as $name => $value){
    $_POST[$name] = mysql_real_escape_string($value);
}

// А потом - следующее (обратите внимание на отсутствие кавычек для GET параметров)
$result = mysql_query("SELECT * FROM ".$_GET['index_country']." WHERE id = ".$_GET['index_to_edit']);

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

skrostislav, 05 Апреля 2010

Комментарии (21)

PHP / Говнокод #2934

+163

1
2
3
4
5
6
7

$birthDay = $elements['birthDay']->getValue();

$d = $birthDay[0].$birthDay[1];
$m = $birthDay[3].$birthDay[4];
$y = $birthDay[6].$birthDay[7].$birthDay[8].$birthDay[9];

$birthDay = mktime(0, 0, 0, (int)$m, (int)$d, (int)$y);

Вот так вот в ПХП 5.2 конвертим дату из dd.mm.yyyy в timestamp

paranoid, 05 Апреля 2010

Комментарии (14)

PHP / Говнокод #2932

+166.2

01
02
03
04
05
06
07
08
09
10
11
12

<?
$fileSelect = array();
$sql  = 'SELECT * FROM bd_file';
$stmt = db_prepare($sql, $file); $res  = db_query($stmt);
while($row = db_fetch_row(res))
{
	if (!is_file(filefolder.'/'.$row[название_файла]))) {
		$fileSelect[$file] = $file;
	}
}
unset($fileSelect);
?>

В добавок к #2929.
Этим образом идет проверка существование документов в панели администратора

MoLe-X, 05 Апреля 2010

Комментарии (3)

PHP / Говнокод #2930

+162.8

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67

<?php 
   function db_connect($host, $user, $pass, $db) {
       global $glink;
       $glink = mysql_connect($host, $user, $pass) or db_give_error(3, '', db_error($glink));
        mysql_select_db($db, $glink) or db_give_error(4, '', db_error($glink));
		mysql_query("SET CHARSET cp1251",$glink);
       return $glink;
   }
   
   function db_query($sql, $log = 0) {
       global $glink;
       $res = mysql_query($sql, $glink) or db_give_error(5, $sql, db_error($glink));
       return $res;
   }
            
   function db_fetch_row($res) {
       $row = empty($res) ? false : mysql_fetch_row($res);
       return $row;
   }
   
   function db_num_rows($res, $stmt = '') {
       $rows = mysql_num_rows($res); 
       if (null === $rows) db_give_error(6, '', db_error($glink));
       return $rows;
   }
  
   function db_num_fields($res, $stmt = '') {
       $fields = empty($res) ? false : mysql_num_fields($res);
       if (null === $fields) db_give_error(7, '', db_error($glink));
       return $fields;
   }
    
   function db_field_name($res, $fld) {
       $name = (empty($res) or !isset($fld)) ? false : mysql_field_name($res, $fld);
       if (false === $name) db_give_error(8, '', db_error($glink));
       return $name;
   }
   
   function db_affected_rows($res) {
       global $glink;
       $rows = mysql_affected_rows($glink); 
       if (null === $rows) db_give_error(9, '', db_error($glink));
       return $rows;
   }

   function db_insert_id($id = 0) {
       $id = mysql_insert_id();
       return ($id) ? $id : 1;
   }
   
   function db_next_id($table, $field, $seqname = 'seq') {
       return 0;
   }
   
   function db_errno($res) {
       return mysql_errno($res);
   }
   
   function db_error($res) {
       return mysql_error();
   }
   
   function db_free_result($res) {
       $res = mysql_free_result($res);
       return $res;
   }
?>

Все из того же проекта (http://govnokod.ru/2929). авторам видимо надоело писать mysql_ и они решили сделать свой аналог db_. При чем смысла от этого полный NULL. Говнокод, он и в Африке говнокод

MoLe-X, 05 Апреля 2010

Комментарии (23)

PHP / Говнокод #2929

+165

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17

$fileSelect = array();
if (is_dir($filefolder)) {
    if ($dh = @opendir($filefolder)) {
        while (($file = readdir($dh)) !== false) {
            if (filetype($filefolder . $file) == "file" and $file != "0.txt") {
                $sql  = 'SELECT * FROM bd_file WHERE file_file = ?';
                $stmt = db_prepare($sql, $file); $res  = db_query($stmt);
                if (!db_num_rows($res)) {
                    $fileSelect[$file] = $file;
                }
            }
        }
    }
    closedir($dh);
}
...
unset($fileSelect);

Сервер медленно умирал, так как данная конструкция вызывалась при каждом удобном случае, будь то добавление нового файла или скачивание. Самое поразительное, что с полученным массивом нечего не делали, а тупо уничтожили

P.S. минимальная стоимость сайта у этой фирмы 3.000.000 рублей

MoLe-X, 05 Апреля 2010

Комментарии (13)

PHP / Говнокод #2927

+159.2

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31

<?php 
include( "dbopen.php" ); 

function ShowTree($ParentID, $lvl) { 

global $link; 
global $lvl; 
$lvl++; 

$sSQL="SELECT id,title,pid FROM catalogue WHERE pid=".$ParentID." ORDER BY title";
$result=mysql_query($sSQL, $link);

if (mysql_num_rows($result) > 0) {
echo("<UL>\n");
while ( $row = mysql_fetch_array($result) ) {
$ID1 = $row["id"];
echo("<LI>\n");
echo("<A HREF=\""."?ID=".$ID1."\">".$row["title"]."</A>"."  \n");
ShowTree($ID1, $lvl); 
$lvl--;
}
echo("</UL>\n");
}

}

ShowTree(0, 0); 

mysql_close($link); 

?>

Построение дерева с запросом в цикле:)))

http://www.codenet.ru/webmast/php/tree.php

ar4ibal, 05 Апреля 2010

Комментарии (19)

PHP / Говнокод #2926

+167.8

1
2
3

if($_GET['admin']=='da'){
	echo 'adminka_true';
}

А самое страшное, что писал это я...

Armanio, 03 Апреля 2010

Комментарии (21)

PHP / Говнокод #2920

+165

1

$result = mysql_query("SELECT `id` FROM `cats` WHERE `id` = (SELECT `pid` FROM `tovary` WHERE `id` = '".$this->id."' LIMIT 1)");

В один запрос получать id не кошерно.

Было найдено в этом же проекте - http://www.govnokod.ru/2897

Rpsl, 02 Апреля 2010

Комментарии (11)

PHP / Говнокод #2917

+166

1

$limit = (isset($this->params['pass'][1]) == 'news')?null:5;

балина, и это работает :D

lener, 02 Апреля 2010

Комментарии (30)

Говнокод: по колено в коде.

PHP / Говнокод #2938

PHP / Говнокод #2935

PHP / Говнокод #2934

PHP / Говнокод #2932

PHP / Говнокод #2930

PHP / Говнокод #2929

PHP / Говнокод #2927

PHP / Говнокод #2926

PHP / Говнокод #2920

PHP / Говнокод #2917