Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > de.comp.lang.php > #4339

Re: PHP speichert manche Dateien mit ?-Suffix ab.

Show all headers | View raw

Tom Wallenfang:

> Am 07.04.2018 um 20:58 schrieb Arno Welzel:
>> Zeige den relevanten Code.
> 
>       <?php
>         $txt = shell_exec('ls -d */');
>         echo $txt;
>         echo "<form action=\"write.php\" method=\"post\">";
>         echo "<input type=\"text\" name=\"ornder\" /><br />";
>         echo "<textarea name=\"input\"
> style=\"width:250px;height:150px\"></textarea><br />";
>         echo "<input type=\"Submit\" value=\"Submit\" />";
>         echo "</form>";
>         $ordner = "test/"; // Normalerweise ebenfalls $_POST[], aber zum
> testen umgeändert

Aua!

Nein - sowas bitte NIEMALS machen! Inahlte von außen, wie $_POST[...]
darf man grundsätzlich NIEMALS direkt als Vorlage für Dateinamen etc.
nutzen.

Ein Angreifer kann dein Script ebenfalls aufrufen und für die
entsprechenden Parameter *beliebigen* Inhalt übergeben, auch solchen,
die Pfadtrenner "/" oder "\" und ".." oder "..." enthalten.

>         $text = $_POST["input"];
>         if($text!="") // bzw. wenn Seite zum ersten mal aufgerufen wird.
>         {
>         $urln = $ordner . "n"; // "test/n" enthält die Nummerdatei bzw.
> den Dateinamen für den neuen Eintrag
> 
>         $readnum = fopen("$urln", "r") or die("Tot.");

Besser:

$readnum = fopen($urln, "r") or die("Tot.");

Es gibt keinen vernünftigen Grund, eine einzelne Variable in
Anführungszeichen zu setzen, wenn man als Parameter den Inhalt der
Variablen nutzen will.

Mir scheint auch, Du zeigst nicht den wirklichen Code. Denn das gezeigte
legt einfach nur die Datei "n" im Ordner "test" an. Wo da noch ein "?"
herkommen soll, ist völlig unklar.

>         $nummerp = fread($readnum,filesize($urln));
>         fclose($readnum);
>         $nummer = $nummerp;
>         $nummerp = $nummerp + 1;
>         $url = $ordner . $nummer; // Der Dateiname
[...]
>         $texts = fopen("$url", "w"); // Schreibt den Text

Auch hier wieder: warum "$url" und nicht einfach $url?

Und wer garantiert, dass der Inhalt der gelesenen Datei wirklich eine
Zahl enthält und $nummer korrekt gesetzt ist?

Ich glaube, Du hast PHP nicht wirklich verstanden. Kann das sein, das Du
noch ein Anfänger bist?


-- 
Arno Welzel
https://arnowelzel.de
https://de-rec-fahrrad.de
http://fahrradzukunft.de

Back to de.comp.lang.php | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

PHP speichert manche Dateien mit ?-Suffix ab. Tom Wallenfang <tom@wallenfang.de> - 2018-04-07 20:14 +0200
  Re: PHP speichert manche Dateien mit ?-Suffix ab. "Christoph M. Becker" <cmbecker69@arcor.de> - 2018-04-07 20:34 +0200
    Re: PHP speichert manche Dateien mit ?-Suffix ab. Tom Wallenfang <tom@wallenfang.de> - 2018-04-08 15:36 +0200
      Re: PHP speichert manche Dateien mit ?-Suffix ab. "Christoph M. Becker" <cmbecker69@arcor.de> - 2018-04-08 16:40 +0200
  Re: PHP speichert manche Dateien mit ?-Suffix ab. Arno Welzel <usenet@arnowelzel.de> - 2018-04-07 20:58 +0200
    Re: PHP speichert manche Dateien mit ?-Suffix ab. Tom Wallenfang <tom@wallenfang.de> - 2018-04-08 15:29 +0200
      Re: PHP speichert manche Dateien mit ?-Suffix ab. "Christoph M. Becker" <cmbecker69@arcor.de> - 2018-04-08 16:41 +0200
        Re: PHP speichert manche Dateien mit ?-Suffix ab. Tom Wallenfang <tom@wallenfang.de> - 2018-04-08 18:09 +0200
          Re: PHP speichert manche Dateien mit ?-Suffix ab. "Christoph M. Becker" <cmbecker69@arcor.de> - 2018-04-08 18:19 +0200
      Re: PHP speichert manche Dateien mit ?-Suffix ab. Arno Welzel <usenet@arnowelzel.de> - 2018-04-08 18:05 +0200
        Re: PHP speichert manche Dateien mit ?-Suffix ab. "Christoph M. Becker" <cmbecker69@arcor.de> - 2018-04-08 18:17 +0200
          Re: PHP speichert manche Dateien mit ?-Suffix ab. Thomas 'PointedEars' Lahn <PointedEars@web.de> - 2018-04-08 19:55 +0200
            Re: PHP speichert manche Dateien mit ?-Suffix ab. "Christoph M. Becker" <cmbecker69@arcor.de> - 2018-04-08 20:35 +0200
  Re: PHP speichert manche Dateien mit ?-Suffix ab. Claus Reibenstein <4spamersonly@kabelmail.de> - 2018-04-07 21:20 +0200
  Re: PHP speichert manche Dateien mit ?-Suffix ab. Stefan+Usenet@Froehlich.Priv.at (Stefan Froehlich) - 2018-04-08 06:03 +0000

csiph-web