Pulsante cancella immediato

[badaze]

Hai questa query : DELETE FROM Ingredienti WHERE ID='$delete'

Se $_GET['id'] = "0' or 1=1 --"


Ti dà : DELETE FROM Ingredienti WHERE ID='0' or 1=1 --'


Poi per ottenere ció basta creare un link :
<a href='ingredienti.php?id="0' or 1=1 --">clicca</a>

[FioreFix]

Hai questa query : DELETE FROM Ingredienti WHERE ID='$delete'

Se $_GET['id'] = "0' or 1=1 --"


Ti dà : DELETE FROM Ingredienti WHERE ID='0' or 1=1 --'


Poi per ottenere ció basta creare un link :
<a href='ingredienti.php?id="0' or 1=1 --">clicca</a>

Non capisco bene questa cosa, ho provato a mettere lo 0 o l' 1=1, non succede nulla.

Comunque ora funziona, puoi verificarlo nel link che ho messo nel primo post (lo smile), inserisci e cancella pure ciò che vuoi, è solo un test.

Quello che non ho ben capito, è il discorso sicurezza, mettiamo che sia professionale.

[badaze]

Mi ero sbagliato su un carattere.
...
ti ho mandato un mp.

Discorso sicurezza. Tutti i dati che sono inviati devono essere controllati prima di essere elaborati.

[FioreFix]

Grazie, questo è un problema dalla barra degli indirizzi, non dalla pagina in se, il controllo, come si fa?
Con una seconda pagina immagino, usando POST al posto di GET, non credo però, o in qualche altro modo?

[badaze]

Il problema sta nella pagina non nell'indirizzo. Non puoi impedire a nessuno di scrivere una url e di eseguirla. Quindi devi controllare i dati che ti sono inviati; che lo siano tramite get o post. Poi per i metodi, cerca in internet o qui ce ne sono tanti.

[FioreFix]

Il problema sta nella pagina non nell'indirizzo. Non puoi impedire a nessuno di scrivere una url e di eseguirla. Quindi devi controllare i dati che ti sono inviati; che lo siano tramite get o post. Poi per i metodi, cerca in internet o qui ce ne sono tanti.

Onestamente guide non ne ho trovate tante, quella forse più vicina al mio caso è questa, che però ho capito in parte.
Potresti cortesemente spiegarmi meglio come funziona e come si applica?
Ho fatto un po' di prove, logicamente cambiando l'SQL, ma utilizzando quel:

?id=0 or 1=1 --

viene sempre cancellato tutto.

Grazie!

[badaze]

Devi controllare che il dato che ti è inviato combaci con i tuoi requisiti. Per esempio nell'id aspetti un numero. Quindi devi controllare che il dato passato è un numero.

Esempio :

if (! isset($_GET['id'])) { 
  die("errore 1");
} else {
  $id = $_GET['id'];
  if (! preg_match("/^[0-9]+$/",$id)) { 
      die("errore 2");
  }
  // se arriva qui allora id corrisponde ad un numero.
} 

Ovviamente l'esempio è solo una dritta. Tocca a te vedere cosa fare nel caso in cui c'è un errore.

[FioreFix]

Devi controllare che il dato che ti è inviato combaci con i tuoi requisiti. Per esempio nell'id aspetti un numero. Quindi devi controllare che il dato passato è un numero.

Esempio :

if (! isset($_GET['id'])) { 
  die("errore 1");
} else {
  $id = $_GET['id'];
  if (! preg_match("/^[0-9]+$/",$id)) { 
      die("errore 2");
  }
  // se arriva qui allora id corrisponde ad un numero.
} 

Ovviamente l'esempio è solo una dritta. Tocca a te vedere cosa fare nel caso in cui c'è un errore.

Credevo servisse il

mysql_real_escape_string() 

quindi ok.
L'errore 1 è in caso la variabile sia nulla, avevo già precluso quell'errore no?
Mentre l'errore 2 funziona alla grande, direi che ora funziona tutto.
Il rischio era solo quello?

Grazie veramente tante, mi sei stato di grandissimo aiuto!

[badaze]

Mysql_real_escape_string serve eccome !

[FioreFix]

Mysql_real_escape_string serve eccome !

Correggimi se non ho capito bene:
Quella funzione verifica i caratteri immessi, io invece nella pagina, accettavo solo lettere, quelli accentati e spazio:

function CheckName($checkname)
    {
    $charsok = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZàèìòù ";
    for($x=0; $x<strlen($checkname); $x++)
        {
        $string=substr($checkname, $x, 1); 
        $charsk=strrpos($charsok, $string);    
        if ($charsk===false)
            {
            return(false);
            }
        }
    return(true);
    } 

Quindi qualsiasi altro carattere veniva bloccato.
Alla fine è la stessa cosa, no?