Sicherheit im VMS

Druckbare Version

28.01.2007, 14:12
Pantau

Sicherheit im VMS

1.
Bei der Sicherheitslücke handelt es sich um einen Fehler in der Nickpage-Datei.

Du musst die Datei content/nickpage.php prüfen;
dort sollte sich im oberen Teil diese oder eine ähnliche Zeile befinden:

$id = $_GET['id'];

$id = (int)$_GET['id'];
Das sollte diese Lücke schließen! Herzlichen Glückwunsch!

Bei der Sicherheitslücke handelt es sich um einen Fehler in der Nickpage-Datei.
--------

2.
Bei der Sicherheitslücke handelt es sich um einen Fehler in der Userprofil-Datei.

Du musst die Datei content/konto/userprofil.php prüfen;
dort sollte sich im oberen Teil diese oder eine ähnliche Zeile befinden:

$_POST['max_forced'];

(int)$_POST['max_forced'];
Das sollte diese Lücke schließen! Herzlichen Glückwunsch!

Dies haben 2 Progger bestätigt
20.02.2007, 13:28
MasterG

Hallo,

also ich habe das 1. schon mal geändert. Aber beim 2. bin ich mir noch unsicher. Bei mir kommt das so vor:

Zitat:

if ($_POST['newsletter'] == 0 and $_POST['paidmails'] == 0) $mailstatus = 0;
if ($_POST['newsletter'] == 1 and $_POST['paidmails'] == 0) $mailstatus = 1;
if ($_POST['newsletter'] == 0 and $_POST['paidmails'] == 1) $mailstatus = 2;
if ($_POST['newsletter'] == 1 and $_POST['paidmails'] == 1) $mailstatus = 3;
db_query("UPDATE ".$db_prefix."_emaildaten SET freigabe_fuer = ".$mailstatus.", emailadresse = '".$_POST['emailadresse']."' WHERE uid=".$_SESSION['uid']."");
db_query("UPDATE ".$db_prefix."_userdaten SET max_forced = '".$_POST['max_forced']."' WHERE uid=".$_SESSION['uid']."");
$change .= 'Deine Daten wurden aktualisiert!<br>';

Also nach $_POST['max_forced'] ist kein ; und es ist ja nicht einzeln. Kann ich das bedenkenlos ersetzen durch das (int)$_POST['max_forced']; ?
01.03.2007, 08:06
ElDani

hmmm

das frag ich mich auch gerade...

wie setzt man das dort ein?

Danke schon mal! :)
01.03.2007, 09:22
SebbyPHM

einfach in eine neue Zeile vor dem db_query mit INSERT INTO folgendes einfügen:

Code:

$_POST['max_forced'] = (int)$_POST['max_forced'];
01.03.2007, 10:05
ElDani

hmmm

sorry, bitte, aber
wo bau ich denn die freie Zeile?
und
INSERT INTO ? O.O
nich einfach das StringDingens da rein kopieren?
?(
02.04.2007, 18:55
MasterG

Meinst du das so?

Zitat:

$_POST['max_forced'] = (int)$_POST['max_forced'];
db_query("UPDATE ".$db_prefix."_userdaten SET max_forced = '".$_POST['max_forced']."' WHERE uid=".$_SESSION['uid']."");
02.06.2007, 00:38
TS7

ist das jetzt so richtig wie in dem Post über mir?
23.06.2007, 08:03
jr-coolman

Weis jetzt jemand ob das so korrekt ist wie das in der vorletzten post gemacht wurde?
23.06.2007, 08:52
MK75

So hier muß das, wenn ich nicht irre, bin schon etwas raus aus dem VMS.

PHP-Code:

if ($_POST['newsletter'] == 0 and $_POST['paidmails'] == 0) $mailstatus = 0; if ($_POST['newsletter'] == 1 and $_POST['paidmails'] == 0) $mailstatus = 1; if ($_POST['newsletter'] == 0 and $_POST['paidmails'] == 1) $mailstatus = 2; if ($_POST['newsletter'] == 1 and $_POST['paidmails'] == 1) $mailstatus = 3; db_query("UPDATE ".$db_prefix."_emaildaten SET freigabe_fuer = ".$mailstatus.", emailadresse = '".$_POST['emailadresse']."' WHERE uid=".$_SESSION['uid'].""); $_POST['max_forced'] = (int)$_POST['max_forced'];
23.06.2007, 16:38
ElDani

oki, danke!

Schönes Wochenende! :)
23.06.2007, 16:39
djws

also beide Lösungsvorschläge sind verwendbar.
Es ist egal ob das int.... vor der o.g. db_query kommt oder danach, denn in der angegebenen Query wird $_POST['maxbanner'] ja nicht benutzt, weil da was anderes geupdated wird.

eleganter is die Variante von MK75, weil es das Ganze ein bissl weniger durcheinander bringt als die andere Lösung.
02.07.2007, 18:56
jr-coolman

Zitat:

Original von MK75
So hier muß das, wenn ich nicht irre, bin schon etwas raus aus dem VMS.

PHP-Code:

if ($_POST['newsletter'] == 0 and $_POST['paidmails'] == 0) $mailstatus = 0; if ($_POST['newsletter'] == 1 and $_POST['paidmails'] == 0) $mailstatus = 1; if ($_POST['newsletter'] == 0 and $_POST['paidmails'] == 1) $mailstatus = 2; if ($_POST['newsletter'] == 1 and $_POST['paidmails'] == 1) $mailstatus = 3; db_query("UPDATE ".$db_prefix."_emaildaten SET freigabe_fuer = ".$mailstatus.", emailadresse = '".$_POST['emailadresse']."' WHERE uid=".$_SESSION['uid'].""); $_POST['max_forced'] = (int)$_POST['max_forced'];

Also wenn ich diesen Block so einfüge habe ich den Bug wieder der bei der Orginal Downoad version auch vorhanden ist ab dann lassen sich Vor und nachname nicht mehr ändern.
Mein Block bei dem man vor und zu Name ändern kann sieht so aus:

PHP-Code:

$_POST['vorname'] = addslashes($_POST['vorname']); $_POST['nachname'] = addslashes($_POST['nachname']); if ($_POST['newsletter'] == 0 and $_POST['paidmails'] == 0) $mailstatus = 0; if ($_POST['newsletter'] == 1 and $_POST['paidmails'] == 0) $mailstatus = 1; if ($_POST['newsletter'] == 0 and $_POST['paidmails'] == 1) $mailstatus = 2; if ($_POST['newsletter'] == 1 and $_POST['paidmails'] == 1) $mailstatus = 3; db_query("UPDATE ".$db_prefix."_emaildaten SET freigabe_fuer = ".$mailstatus.", emailadresse = '".$_POST['emailadresse']."' WHERE uid=".$_SESSION['uid'].""); db_query("UPDATE ".$db_prefix."_userdaten SET vorname = '".$_POST['vorname']."', nachname = '".$_POST['nachname']."', max_forced = '".$_POST['max_forced']."' WHERE uid=".$_SESSION['uid']."");

Was muss ich den an dem ändern das der auch Bugfrei ist?
03.07.2007, 09:05
MK75

Zitat:

Original von jr-coolman

Zitat:

Original von MK75
So hier muß das, wenn ich nicht irre, bin schon etwas raus aus dem VMS.

PHP-Code:

if ($_POST['newsletter'] == 0 and $_POST['paidmails'] == 0) $mailstatus = 0; if ($_POST['newsletter'] == 1 and $_POST['paidmails'] == 0) $mailstatus = 1; if ($_POST['newsletter'] == 0 and $_POST['paidmails'] == 1) $mailstatus = 2; if ($_POST['newsletter'] == 1 and $_POST['paidmails'] == 1) $mailstatus = 3; db_query("UPDATE ".$db_prefix."_emaildaten SET freigabe_fuer = ".$mailstatus.", emailadresse = '".$_POST['emailadresse']."' WHERE uid=".$_SESSION['uid'].""); $_POST['max_forced'] = (int)$_POST['max_forced'];

Also wenn ich diesen Block so einfüge habe ich den Bug wieder der bei der Orginal Downoad version auch vorhanden ist ab dann lassen sich Vor und nachname nicht mehr ändern.
Mein Block bei dem man vor und zu Name ändern kann sieht so aus:

PHP-Code:

$_POST['vorname'] = addslashes($_POST['vorname']); $_POST['nachname'] = addslashes($_POST['nachname']); if ($_POST['newsletter'] == 0 and $_POST['paidmails'] == 0) $mailstatus = 0; if ($_POST['newsletter'] == 1 and $_POST['paidmails'] == 0) $mailstatus = 1; if ($_POST['newsletter'] == 0 and $_POST['paidmails'] == 1) $mailstatus = 2; if ($_POST['newsletter'] == 1 and $_POST['paidmails'] == 1) $mailstatus = 3; db_query("UPDATE ".$db_prefix."_emaildaten SET freigabe_fuer = ".$mailstatus.", emailadresse = '".$_POST['emailadresse']."' WHERE uid=".$_SESSION['uid'].""); db_query("UPDATE ".$db_prefix."_userdaten SET vorname = '".$_POST['vorname']."', nachname = '".$_POST['nachname']."', max_forced = '".$_POST['max_forced']."' WHERE uid=".$_SESSION['uid']."");

Was muss ich den an dem ändern das der auch Bugfrei ist?

Danke Dir, aber mal abgesehen davon, wenn ich mich wo anmelde, heiße ich doch immer wie ich heiße? Oder wozu sollte ich meinen Namen ändern können? Wenn ein User seinen Namen änder will, was ja schon etwas sehr selten vorkommt, kann er sich doch an den Betreiber wenden. Kann ja schon mal sein das jemand heiratet, aber mal ehrlich.
03.07.2007, 17:34
jr-coolman

Stimmt.
Aber wenn man das so machen möchte dann würde ich es besser finden wenn man es gleich so macht das der name garnicht änderbar ist.
Und da sind wir wieder bei meinem Problem.
Das habe ich nämlich nicht kapiert wie das geht ich habe die ursprübgliche zeile:

PHP-Code:

<td>Name:</td> <td><input type="text" name="nachname" value="<?=$userdaten['nachname'];?>"></td> </tr>

Genommen und so verändert:

PHP-Code:

<td>Name:</td> <td><?=$userdaten['nachname'];?>"></td> </tr>

Da sie ja so in der richtung ja auch in als Klamm ID anzeige steht.
Der Trick ist das wenn man dann auf "Jetzt ändern" klickt ist die Namenszeile ganz lehr.
Warum das so ist habe ich noch nicht verstanden?
03.07.2007, 18:10
mastermix

ich vermute, weil du, wenn du auf die "Jetzt ändern" seite kommst, keine abfrage der daten mehr gemacht wird und er somit nichts hat, um es einzutragen.
versuchs doch so, mit readonly:

PHP-Code:

<td>Name:</td> <td><input type="text" name="nachname" value="<?=$userdaten['nachname'];?>" readonly="readonly"></td> </tr>

so wirds normal angezeigt, aber trotzdem normal übergeben
03.07.2007, 18:29
jr-coolman

Stimmt so geht es.
Der Rahmen ist dann zwar nach wie vor um den Namen und Vornamen aber ändern lassen sie sich nicht mehr.
03.07.2007, 18:50
mastermix

Zitat:

Original von jr-coolman
Stimmt so geht es.
Der Rahmen ist dann zwar nach wie vor um den Namen und Vornamen aber ändern lassen sie sich nicht mehr.

und das wolltest du ja. ansonsten hat man halt immer das problem, dass die daten irgendwie übergeben werden müssen. bei der klamm id ist das nicht der fall, weil die in einer session gepeichert wird, deswegen ist die auch mit dem angegebenen code auf der nächsten seite anzeigbar
03.07.2007, 19:37
jr-coolman

Ah ja.
Ja das ist so Prima.