BlindSQL-Injection Game-Pool Addon Skandal für VMS 1.x

[Drakor]

Hallo,

ich möchte melden, dass sich in dem Addon eine BlindSQL-Injection Vulnerability befindet. Wieder ist es der Parameter $_POST['id'], anscheinend findet auch hier keinerlei Überprüfung statt.
Andere Lücken in dem Addon sind möglich, ich habe nur diese eine Lücke geblackboxed, den Rest habe ich mir noch nicht angeschaut.

Nachzulesen: http://bettlerlose.de

Wäre nett wenn jemand Skandal hierauf aufmerksam machen kann.

Gruß
Drakor

[j.knopf]

suche:
if (!isset($_POST['buy'])) $_POST['buy']= "";

und füge darunter das hier ein:

$_POST['id']=abs(intval($_POST['id']));

[Drakor]

Dankeschön, waren das alle, die in dem Skript existierten ?

wieso arbeiten die Leute hier eigentlich nicht mit einer diff ähnlichen Software ?

Gruß
Drakor

[jpwfour]

...
wieso arbeiten die Leute hier eigentlich nicht mit einer diff ähnlichen Software ?
...

Wen meinst du damit, und was sollte das bringen?

Dadurch beheben sich Bugs auch nicht von selbst und ein Großteil der "Kunden" hat schon Probleme, Dateien korrekt via FTP hochzuladen, da wäre jeder diff-ähnliche Ansatz wohl zu viel verlangt (geschweige denn können sicher auch viele der im Klammlose-Bereich tätigen "Programmierer" nicht richtig mit solchen Tools umgehen).

[Drakor]

Wieso, durch das diff wirds doch noch einfacher ?
Dann braucht der User das Addon nurnoch hochladen und der Progger nur ne .diff als Patch bereitstellen.

(wobei das diff Format wohl nicht ganz tauglich ist, aber aus anderen Gründen)

[Sebmaster]

Pff, du hast noch nie für Addons Support geben müssen, oder?

Da muss für .diff wieder Software installiert werden, die Leute haben keine Ahnung wie sie das bedienen und regen sich drüber auf, wieso das nicht wie bei allen anderen Addons ganz einfach sein kann...

[breaker]

suche:
if (!isset($_POST['buy'])) $_POST['buy']= "";

und füge darunter das hier ein:

$_POST['id']=abs(intval($_POST['id']));

Was genau soll das bringen?

Das:

if (!isset($_POST['buy']))    $_POST['buy']= ""; 

stellt nur fest ob "buy" im _POST-Array vorhanden ist, ist es nicht vorhanden wird es ja noch vernünftig deklariert, ist es aber vorhanden, wird es nicht mal auf Inhalt geprüft.

Vernünftiger wäre hier sowas (wenn "buy" ein string ist):

$buy = (!isset($_POST['buy']) ? '' : addslashes($_POST['buy']));

Wobei das ganze natürlich auch über andere Funktionen escapt werden kann/darf

Achso.das ist auch kein SQL, sondern ein PHP-Injection ;-)

[Aradiv]

$_POST['id']=abs(intval($_POST['id'])); 

stellt sicher das $_POST['id'] eine positive Ganzzahl ist.

[neoplacer]

Was genau soll das bringen?

Das:

if (!isset($_POST['buy']))    $_POST['buy']= ""; 

stellt nur fest ob "buy" im _POST-Array vorhanden ist, ist es nicht vorhanden wird es ja noch vernünftig deklariert, ist es aber vorhanden, wird es nicht mal auf Inhalt geprüft.

Vernünftiger wäre hier sowas (wenn "buy" ein string ist):

$buy = (!isset($_POST['buy']) ? '' : addslashes($_POST['buy']));

Wobei das ganze natürlich auch über andere Funktionen escapt werden kann/darf

Achso.das ist auch kein SQL, sondern ein PHP-Injection ;-)

addslashes Ist nicht sicher lässt sich sehr einfach aus hebeln..
Wenn du weißt das da nur eine Ganzzahl kommen darf, dann sollte man das auch darauf Prüfen.
Und wenn du die ID so wie sie in die $var gespeichert ist an deine DB via aufruf übergibst ist es eine SQL Injektion

PHP wäre wenn du Include($_POST["pagename"].".php"); machen würdest
Denn pagename= http://meineurl.tls/bösesscript führt das dann aus.