Wątek: [SQL Injection] market.php (Wszystkie podstrony)

1. Opis błędu

Brak filtracji $_GET['lista'] i $_GET['limit']

2. Zagrożenie

Błąd umożliwiający atak SQL Injection.

3. Rozwiązanie

[OTWÓRZ]
amarket.php
[ZNAJDŹ]

Kod php:

    if ($_GET['limit'] < $oferty) 
    { 


[DODAJ PO]

Kod php:

        $_GET['limit'] = intval($_GET['limit']);
        if (!in_array($_GET['lista'], array('id', 'type', 'number', 'amount', 'cost', 'seller'))) error(ERROR); 


[OTWÓRZ]
hmarket.php
[ZNAJDŹ]

Kod php:

    if ($_GET['limit'] < $oferty) 
    { 


[DODAJ PO]

Kod php:

        $_GET['limit'] = intval($_GET['limit']);
        if (!in_array($_GET['lista'], array('id', 'nazwa', 'ilosc', 'cost', 'seller'))) error(ERROR); 


[OTWÓRZ]
imarket.php
[ZNAJDŹ]

Kod php:

    if ($_GET['limit'] < $przed) 
    { 


[DODAJ PO]

Kod php:

        $_GET['limit'] = intval($_GET['limit']);
        if (!in_array($_GET['lista'], array('id', 'name', 'power', 'wt', 'szyb', 'zr', 'minlev', 'amount', 'cost', 'owner'))) error(ERROR); 


[OTWÓRZ]
mmarket.php
[ZNAJDŹ]

Kod php:

    if ($_GET['limit'] < $przed) 
    { 


[DODAJ PO]

Kod php:

        $_GET['limit'] = intval($_GET['limit']);
        if (!in_array($_GET['lista'], array('id', 'name', 'efect', 'amount', 'cost', 'owner'))) error(ERROR); 


[OTWÓRZ]
pmarket.php
[ZNAJDŹ]

Kod php:

    if ($_GET['limit'] < $oferty) 
    { 


[DODAJ PO]

Kod php:

        $_GET['limit'] = intval($_GET['limit']);
        if (!in_array($_GET['lista'], array('nazwa', 'ilosc', 'cost', 'seller'))) error(ERROR); 


[OTWÓRZ]
rmarket.php
[ZNAJDŹ]

Kod php:

    if ($_GET['limit'] < $przed) 
    { 


[DODAJ PO]

Kod php:

        $_GET['limit'] = intval($_GET['limit']);
        if (!in_array($_GET['lista'], array('id', 'name', 'power', 'milev',  'amount', 'cost', 'owner'))) error(ERROR); 


Chyba jednak trzeba zmienic to is_int na is_numeric poniewaz przy wklejeniu is_int odrazu na wejsciu pisze Zapomnij o tym ! Chyba ze cos zle zrobilem
imarket.php

Kod php:

 if (!isset($_GET['limit']))
    {
        $_GET['limit'] = 0;
    }
    if (!is_int($_GET['limit'])) error(ERROR);
    if (!in_array($_GET['lista'], array('id', 'name', 'power', 'wt', 'szyb', 'zr', 'minlev', 'amount', 'cost', 'owner'))) error(ERROR);  
    $smarty -> assign(array("Tname" => T_NAME,
        "Tpower" => T_POWER,
        "Tcost" => T_COST,
        "Tseller" => T_SELLER,
        "Tdur" => T_DUR,
        "Tspeed" => T_SPEED,
        "Tagi" => T_AGI,
        "Tamount" => T_AMOUNT,
        "Tlevel" => T_LEVEL,
        "Viewinfo" => VIEW_INFO,
        "Toptions" => T_OPTIONS)); 


o kurde nie zauważyłem, ze w paru miejscach jest przed issetem. już naprawiam. Havard, popraw teraz wg. nowej instrukcji.

To jeszcze w house.php tak jest

// To nie temat od house.php
// McFly