Fjern skadelige ting før insert i db

Hov, der sneg sig vidst lige en fejl ind, men i kan nok se hvad jeg mener alligevel :)

Jeg vil mene du er pænt sikret der! :o)

Ja det vil jeg også selv, men jeg er ikke sikker på om det kan bruges i praksis.
For eksempel kan en jo forsøge at vise noget PHP. Vil det i så fald kunne læses i "den anden ende"?

Måske det er bedre sådan her:

function clean($zap){
    $zap = trim($zap);
    $zap = htmlentities($zap, ENT_NOQUOTES);
    $zap = strip_tags($zap);
    $zap = mysql_real_escape_string($zap);
    return $zap;
}

Bemærk den ændrede rækkefølge.
Eller er det overkill at bruge både strip_tags og htmlentities?
Jeg har ikke den mindste idé om hvor effektive de er.

Jeg bruger personligt selv kun mysql_real_escape_string()

addslashes() kan også bruges istedet for mysql_real_escape_string() (da sidstnævnte kræver at mysql virker.. dvs. dit system ville være ubrugeligt i et ikke-mysql-miljø)

det anbefales at du bruge strip_tags() og htmlentitie() ved OUTPUT istedet.
det er faktisk kun addslashes() som er nødvendig før DB insert.

(og så alm. validering på ALLE informationer , men det er jo genneralt procedure).

mysql_real_escape_string()
Vil man beskytte noget som ikke skal i DB?

Mit "system" kommer aldrig til at køre på andet end MySQL.

Du kan ikke sammenligne addslashes og mysql_real_escape_strings.
addslashes ændre kun på ' " \ og NUL hvorimod mysql_real_escape_string også gør det ved \x00, \n, \r og \x1a

Hvis du laver din funktion til:
function clean($zap){
    $zap = trim($zap);
    $zap = strip_tags($zap);
    $zap = htmlentities($zap, ENT_NOQUOTES);
    $zap = mysql_real_escape_string($zap);
    return $zap;
}

Så vil du bare godt beskyttet!
bemærk jeg rykkede strip_tags() op over htmlentities()

Men som jeg spurgte om før: Hvis strip_tags er øverst, så er det vel ikke muligt for en bruger at tilføje fx PHP kode uden det hele bliver fjernet?

Det har du muligvis ret i! :o)

Nå, det må jeg teste. Tak for hjælpen. Smider du et svar?

Hvis du synes, så skidt da! :o)

Takker for point!