Error, is dit veilig?

Als ik dit in me URL ding zet: http://localhost/nieuws?id=

komt er deze error:

Warning: Invalid argument supplied for foreach() in C:xampphtdocs
ieuws.php on line 33

Is dit veilig tegen SQL Injections etc? Hoe kan ik deze fout oplossen?

Line 33:

[COLOR= #0000BB]<?php[/COLOR] 
[URL="http://www.php.net/foreach"]foreach[/URL] [COLOR= #007700]([/COLOR][COLOR= #0000BB]$news_row[/COLOR] [COLOR= #007700]as &[/COLOR][COLOR= #0000BB]$value[/COLOR][COLOR= #007700])[/COLOR]
?>

Foreach verwacht een array, die jij niet geeft.

Ergens in je mysql fout-afhandeling gaat het dus al fout omdat je niet checkt óf je rijen hebt.

Rik schreef:

Foreach verwacht een array, die jij niet geeft.

Ergens in je mysql fout-afhandeling gaat het dus al fout omdat je niet checkt óf je rijen hebt.

Hmm. snap ik niet echt

[COLOR= #0000BB]<?php
$news_row[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$db[/COLOR][COLOR= #007700]->[/COLOR]fetch_assoc[COLOR= #007700]([/COLOR][COLOR= #0000BB]$db[/COLOR][COLOR= #007700]->[/COLOR]query[COLOR= #007700]([/COLOR][COLOR= #DD0000]"SELECT * FROM "[/COLOR][COLOR= #007700].[/COLOR]PREFIX[COLOR= #007700].[/COLOR][COLOR= #DD0000]"news WHERE id = "[/COLOR][COLOR= #007700].[/COLOR] [URL="http://www.php.net/mysql_real_escape_string"]mysql_real_escape_string[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$id[/COLOR][COLOR= #007700]) .[/COLOR][COLOR= #DD0000]" LIMIT 1"[/COLOR][COLOR= #007700]));[/COLOR]
[URL="http://www.php.net/foreach"]foreach[/URL] [COLOR= #007700]([/COLOR][COLOR= #0000BB]$news_row[/COLOR] [COLOR= #007700]as &[/COLOR][COLOR= #0000BB]$value[/COLOR][COLOR= #007700]) {[/COLOR]
    [COLOR= #0000BB]$value[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$input[/COLOR][COLOR= #007700]->[/COLOR]HoloText[COLOR= #007700]([/COLOR][COLOR= #0000BB]$value[/COLOR][COLOR= #007700],[/COLOR] true[COLOR= #007700]);
}[/COLOR]
?>

stukje meer code.

Waar check je of je mysql results teruggeeft?
Nergens.

Rik schreef:

Waar check je of je mysql results teruggeeft?
Nergens.

Hmm.

Maar als ik die weg haal heb ik die error niet..

Is het veilig?

... Je luistert echt niet naar wat ik zeg hé?

En ja, het is verder wel veilig voor SQL injection

Wat Rik bedoeld is:

[COLOR= #0000BB]<?php
$news_row[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$db[/COLOR][COLOR= #007700]->[/COLOR]fetch_assoc[COLOR= #007700]([/COLOR][COLOR= #0000BB]$db[/COLOR][COLOR= #007700]->[/COLOR]query[COLOR= #007700]([/COLOR][COLOR= #DD0000]"SELECT * FROM "[/COLOR][COLOR= #007700].[/COLOR]PREFIX[COLOR= #007700].[/COLOR][COLOR= #DD0000]"news WHERE id = "[/COLOR][COLOR= #007700].[/COLOR] [URL="http://www.php.net/mysql_real_escape_string"]mysql_real_escape_string[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$id[/COLOR][COLOR= #007700]) .[/COLOR][COLOR= #DD0000]" LIMIT 1"[/COLOR][COLOR= #007700]));[/COLOR]
[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/mysql_num_rows"]mysql_num_rows[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$news_row[/COLOR][COLOR= #007700]) >[/COLOR] [COLOR= #0000BB]0[/COLOR][COLOR= #007700])
{[/COLOR]
    [URL="http://www.php.net/foreach"]foreach[/URL] [COLOR= #007700]([/COLOR][COLOR= #0000BB]$news_row[/COLOR] [COLOR= #007700]as &[/COLOR][COLOR= #0000BB]$value[/COLOR][COLOR= #007700]) {[/COLOR]
        [COLOR= #0000BB]$value[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$input[/COLOR][COLOR= #007700]->[/COLOR]HoloText[COLOR= #007700]([/COLOR][COLOR= #0000BB]$value[/COLOR][COLOR= #007700],[/COLOR] true[COLOR= #007700]);
    }
}[/COLOR]
?>

En dan kan je het beter zo doen:

[COLOR= #0000BB]<?php
$news_row[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$db[/COLOR][COLOR= #007700]->[/COLOR]fetch_assoc[COLOR= #007700]([/COLOR][COLOR= #0000BB]$db[/COLOR][COLOR= #007700]->[/COLOR]query[COLOR= #007700]([/COLOR][COLOR= #DD0000]"SELECT * FROM "[/COLOR][COLOR= #007700].[/COLOR]PREFIX[COLOR= #007700].[/COLOR][COLOR= #DD0000]"news WHERE id = "[/COLOR][COLOR= #007700].[/COLOR] [URL="http://www.php.net/mysql_real_escape_string"]mysql_real_escape_string[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$id[/COLOR][COLOR= #007700]) .[/COLOR][COLOR= #DD0000]" LIMIT 1"[/COLOR][COLOR= #007700]));[/COLOR]
[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/mysql_num_rows"]mysql_num_rows[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$news_row[/COLOR][COLOR= #007700]) >[/COLOR] [COLOR= #0000BB]0[/COLOR][COLOR= #007700])
{[/COLOR]
    [URL="http://www.php.net/foreach"]foreach[/URL] [COLOR= #007700]([/COLOR][COLOR= #0000BB]$news_row[/COLOR] [COLOR= #007700]as &[/COLOR][COLOR= #0000BB]$value[/COLOR][COLOR= #007700]) {[/COLOR]
        [COLOR= #0000BB]$value[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$input[/COLOR][COLOR= #007700]->[/COLOR]HoloText[COLOR= #007700]([/COLOR][COLOR= #0000BB]$value[/COLOR][COLOR= #007700],[/COLOR] true[COLOR= #007700]);
    }
}[/COLOR]
[URL="http://www.php.net/else"]else[/URL]
[COLOR= #007700]{[/COLOR]
    [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]Nieuws bericht niet gevonden![/COLOR][COLOR= #007700];[/COLOR] [COLOR= #FF8000]// aanpassen naar wat er niet gevonden kon worden[/COLOR]
[COLOR= #007700]}[/COLOR]
?>

^ Fijn dat het werk weer gedaan word voor de TS zodat hij de volgende keer weer niet zelf zoekt...

Rik schreef:

^ Fijn dat het werk weer gedaan word voor de TS zodat hij de volgende keer weer niet zelf zoekt.

De volgende keer weet hij het dan, en als hij dan nogmaals vraagt zeg ik ook niets meer..
En ik ben wel met je eens dat hij beter naar jou had kunnen luisteren en even Googlen..

Bedankt.

toevoeging op 18 april 2012, 15:50:04:
Ik zou voortaan eerst Google`n
toevoeging op 18 april 2012, 15:50:04:
Ik zou voortaan eerst Google`n