Ist das Script so sicher?

[fanrpg]

Hier mal das Script: http://www.fanrpg.com/code.php?id=1

Ich würde gerne wissen ob das sicher ist...
Nicht das einer irgendwie ne .php Datei hochlädt und so mein Forum hacken oder meinetwegen auch anders.

Weiss das jemand zufällig und könnte mal kurz drüber schauen?

[rkern]

Code: Alles auswählen

$file = fopen('./scripts/script_'.$new_id.'.txt', 'w'); 

uninitialisierte Vairable $new_id: wenn register_globals gesetzt ist, kann man damit $new_id von aussen setzen und irgendwo in dein Filesystem schreiben.

mfg Ralph

[fanrpg]

Danke
Habe jetzt mal einen allgemeinen Schutz vor register_globals eingebaut das müsste doch reichen?

[S2B]

Ähm, und was machst du, wenn das nie ausgeführt wird:

Code: Alles auswählen

        if( !in_array($i, $lines_in_file) )
        {
            $new_id = $i;
            break;
        }

Dann ist $new_id undefiniert, er wirft einen Fehler und öffnet script_.txt.

Generell gilt: Ein genereller Schutz vor register_globals befreit dich nicht vom Vordefinieren von Variablen, wenn du guten Code schreiben willst.

Edit:
Außerdem funktioniert der Schutz soweit ich das sehe nicht:

Code: Alles auswählen

if( $name == $$name )

Müsste das erste $name nicht $value heißen?

[fanrpg]

in $value steht doch dann der Inhalt von $name.
Aber ich suche ja den Namen von $name also in dem Fall $$name.
Der name der variable wird dann die variable so genannt.
Aber hast recht hab einen Fehler gefunden

Dann ist $new_id undefiniert, er wirft einen Fehler und öffnet script_.txt.

Ich definiere es dann einfach und gut ist.
Aber warum sollte der das nicht ausführen? steht ja nicht wirklich in einer if Klammer und 1000 Scripts muss man erstmal hinkriegen

[S2B]

in $value steht doch dann der Inhalt von $name.
Aber ich suche ja den Namen von $name also in dem Fall $$name.
Der name der variable wird dann die variable so genannt.
Aber hast recht hab einen Fehler gefunden

Moment...

Code: Alles auswählen

$_GET = array(
	'new_code' => 'Mein Code');

foreach ($_GET as $name => $value)
{
	/*
		- $name enthält 'new_code'
		- $value enthält 'Mein Code'
		- $$name ist die Variable mit dem Namen $name = 'new_code', sprich $new_code
		- $$name enthält, wenn register_globals auf on steht, den Wert der GET-Variable,
		  also $value oder auch 'Mein Code'
	*/
	if ($value == $$name)
	{
		unset($$name);
	}
}

Aber warum sollte der das nicht ausführen? steht ja nicht wirklich in einer if Klammer und 1000 Scripts muss man erstmal hinkriegen

Ich meinte das, was in der if-Abfrage steht, sprich das Definieren der Variable $new_id.

[fanrpg]

in $value steht doch dann der Inhalt von $name.
Aber ich suche ja den Namen von $name also in dem Fall $$name.
Der name der variable wird dann die variable so genannt.
Aber hast recht hab einen Fehler gefunden

Moment...

Code: Alles auswählen

$_GET = array(
	'new_code' => 'Mein Code');

foreach ($_GET as $name => $value)
{
	/*
		- $name enthält 'new_code'
		- $value enthält 'Mein Code'
		- $$name ist die Variable mit dem Namen $name = 'new_code', sprich $new_code
		- $$name enthält, wenn register_globals auf on steht, den Wert der GET-Variable,
		  also $value oder auch 'Mein Code'
	*/
	if ($value == $$name)
	{
		unset($$name);
	}
}

Aber warum sollte der das nicht ausführen? steht ja nicht wirklich in einer if Klammer und 1000 Scripts muss man erstmal hinkriegen

Ich meinte das, was in der if-Abfrage steht, sprich das Definieren der Variable $new_id.

Hmm hast recht, werde das mal ändern.