ähh, will erstma kurz sagen das das Script wirklich unsicher und unsauber ist...
Die Bewertung beruht sich nur auf den Content-Bereich
Nach den vielen Fehlern habe ich keine Lust mehr durch den Adminbereich zu gehen
Quelltextbewertung:
Kommentar:
Der Installationsvorgang wird dem Nutzer erklärt, jedoch könnte es Missverständnisse geben. Die Ordnerstrucktur ist vorhanden, jedoch nicht, wie vorgesehen, im Ordner
Upload. Es wird nicht der von Ilch gesetzte PREFIX benutzt, sondern \'.DBPREF.\'. Es wird darauf hingewiesen, dass die Installationsdateien gelöscht werden sollten.
Die Dateien sind nicht immer in den richtigen Ordnern! Templates sind teilweise im contents/cupscript Das Modul wird in die Datenbanktabelle "PREFIX_modules" eingetragen und ist somit über den Adminbereich erreichbar. Es ist ein Icon für den Adminbereich vorhanden. Das Modul lässt sich somit leicht wiederfinden.
Es wird nicht immer die Funktion
db() benutzt. Die Dateien verwenden
redirect() statt wd(). PHP-Funktionen, von denen Ilch eine eigene hat. Das Modul nutzt vorzugsweise Templates. Der Quelltext ist leicht anpassbar. Die Ilch-Klasse wird nicht verwendet
Es ist möglich, das Modul ohne index.php direkt aufzurufen! ALLE PHP DATEIEN SIND DIREKT AUFRUFBAR !!!
In den Dateien siehe Anmerkung unten wird ein Post/GET-Array ungefiltert an eine Datenbankanweisung übertragen.
In den Dateien siehe Anmerkung unten wird ein Post/GET-Array ungefiltert ausgegeben.
In den Dateien die Ilch-Menü-Klasse wird nicht verwendet und $_GET[] wird nicht escaped wird ein MENU-Array ungefiltert an eine Datenbankanweisung übertragen.
In den Dateien Siehe Anmerkung unten wird ein MENU-Array ungefiltert ausgegeben.
Die Dateien
include/contents/*.php geben folgende Warnungen zurück:
Notice: Undefined variable: border in C:\\xampp\\htdocs\\shop\\include\\contents\\cups.php(202) : eval()\'d code on line 12
Notice: Undefined variable: leaderclans in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmenu.php on line 36
Notice: Undefined variable: border in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmenu.php(208) : eval()\'d code on line 2
Notice: Undefined variable: bghead in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmenu.php(208) : eval()\'d code on line 6
Notice: Undefined variable: pagebg in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmenu.php(208) : eval()\'d code on line 8
Notice: Undefined variable: border in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmenu.php(208) : eval()\'d code on line 12
Notice: Undefined variable: userteam in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmenu.php(208) : eval()\'d code on line 14
Notice: Undefined variable: border in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmenu.php(208) : eval()\'d code on line 15
Notice: Undefined variable: bghead in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmenu.php(208) : eval()\'d code on line 26
Notice: Undefined variable: pagebg in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmenu.php(208) : eval()\'d code on line 28
Notice: Undefined variable: cups in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmenu.php(208) : eval()\'d code on line 31
Notice: Undefined variable: border in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmenu.php(208) : eval()\'d code on line 43
Notice: Undefined variable: pagebg in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmenu.php(208) : eval()\'d code on line 45
Notice: Undefined variable: onecup in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmenu.php(208) : eval()\'d code on line 46
Notice: Undefined index: cupID in C:\\xampp\\htdocs\\shop\\include\\contents\\cupmatches.php on line 14
Im Script sind schwerwiegende Fehler, mitdenen das Modul nicht vollständig läuft!
/contents/installcup.php
Verwendet php-short-tags
<? statt
<?php und ist dadurch nicht überall kompatibel
(zeile 1 und 325)
/contents/*.php
verwendet manchmal prefix_ und manchmal \'.DBPREF.\' jedoch meistens nicht die Ilch-DB-klasse
POST-Variablen werden unescaped in die Datenbank geschrieben (#54, #56, #138 u.v.m.)
die Ilch-Template-Klasse wird nicht verwendet
WHERE userID = \'$userID\' ->
WHERE clanID = \'$clanID\' ->
-> FEHLER dies wird als String und nicht als Variable behandelt -> \'.$userID.\'
Die Ilch-Menü-Klasse wird nicht verwendet und $_GET[] wird ungefiltert weitergegeben
Eigentlich wollte ich jede Datei einzeln auflisten, aber das wäre zu viel
Das ganze wiederholt sich in jeder Datei
Den Adminbereich teste ich hier nicht mehr weiter (ist auch nicht mehr nötig)
Das Modul ist so nicht akzeptabel
(auch wenns funktionieren sollte, hab nur den Code durchgeschaut und nicht installieren)
Generiert mit dem Modulbewerter 1.0
Zuletzt modifiziert von GeCk0 am 22.11.2009 - 11:26:08
Alle Module von mir sind nun unter php-gecko.de erreichbar