hallo zusammen,
ich versuche $id aus einer txt datei zu vergleiche, aber das klappt bei mir nicht.
im augenblick funktioniert es so bei mir:
if ($id !== "1" && $id !== "2" && $id !== "3") die("id not used!");ich möchte aber, wenn die $id in der txt datei nicht vorhanden ist, dass script beendet wird, mit einer fehlermeldung.
kann mir da jemand bei weiter helfen?
mfg: micha
if ($id <1 
$id>3)
die("id not used!");
währe die einfachere Lösung.
aber die frage ist jetzt, wenn das ganze nicht beendet wird, wie ist dann die id ?! mach doch einfach ne debug Ausgabe.
Hallo GreenRover,
erst mal danke für deine Bemühung, aber ich wollte das die Übergebende ID auch Tatsächlich in data.txt steht. Ist dies nicht der Fall, dann Abbruch mit Fehlermeldung.
So sieht der Auszug aus meinem Script zum einlesen des Counters bis jetzt aus:
$id = isset($_GET["id"]) ? $_GET["id"] : "";
if (preg_match("/[^0-9]+/",$id)) die("Hacking attempt!");
if ($id <1 || $id>3) die("ID: [i]$id[/i] not used!");
$dbfile = "data.txt";
$basedir = "/srv/www/htdocs/web/files/";
$dat = fopen($dbfile, "r");
while($array = fgetcsv($dat, 1000, "|"))
{
if($array[0] == $id)
{
$switch = $array[1];
$digit = $array[2];
$digit++;
}
else
{
$erg[] = "$array[0]|$array[1]|$array[2]\n";
}
}
fclose($dat);<?
error_reporting(E_ALL);
$id = isset($_GET["id"]) ? $_GET["id"] : "";
if (preg_match("/[^0-9]+/",$id)) die("Hacking attempt!");
if ($id <1 || $id>3) die("ID: [i]$id[/i] not used!");
$dbfile = "data.txt";
$basedir = "/srv/www/htdocs/web/files/";
$dat = fopen($dbfile, "r");
while($array = fgetcsv($dat, 1000, "|"))
{
if($array[0] == $id)
{
$switch = $array[1];
$digit = $array[2];
$digit++;
}
else
{
$erg[] = "$array[0]|$array[1]|$array[2]\n";
}
}
fclose($dat);
$dat = fopen($dbfile, "w");
fputs($dat, "$id|$switch|$digit\n");
$count = 0;
while($erg[$count] != "") // Notice: Undefined offset: 2 in /srv/www/htdocs/web/html/download.php on line 30
{
fputs($dat, $erg[$count]);
$count++;
}
fclose($dat);
// exit script for error_reporting(E_ALL);
exit;
$filename = sprintf("%s/%s", $basedir, $switch);
header("Content-Type: application/octet-stream");
$savename = basename($switch);
header("Content-Disposition: attachment; filename=\"$savename\"");
readfile($filename);
?>Habe den debugg Modus aktiviert und im Script ein Kommentar geschrieben. Was kann ich tun, um dieses Script zu Perfektionieren?
Habe dieses Script mehr oder weniger zusammen gestellt, aus verschiedenen Tuts und Script snippets.
Mein Behühen ist, dass dieses Script sicher und Korrekt Funktioniert.
ich wuerde z.B.
if (preg_match("/[^0-9]+/",$id)) die("Hacking attempt!");
durch
if(!is_numeric($id)) die("Hacking attempt!");
ersetzen. Ist allerdings nur eine Schoenheitsoperation.
Sicher ist es, da es keine unkontrollierten Eingaben hat, und man mit Ziffern hoechstens die RAM fuellen kann 
Um gegen jeden GAU gesichert zu sein, kann man noch die eine oder andere Zusatzfunktion einbauen, was allerdings nicht notwendig ist.
hi d4f,
danke für den Tipp, hast du denn eine Idee wie ich das Realisieren kann? Das die ID in:
$dbfile = "data.txt";
vor dem Download abgefragt wird? Im Moment wird die ID so abgefragt:
if ($id <1 || $id>3) die("ID: [i]$id[/i] not used!");ich verstehe jetzt nicht genau was du meinst?
Wie vor dem Download? Das Script wird doch komplett vor dem Download geladen oder nicht? *confused*
Hi d4f,
In der data.txt steht:
1|datei.exe|0
2|datei2.exe|0
3|datei3.exe|0Rufe ich eine ID 4 auf, wird eine neue Zeile in data.txt geschrieben:
||Es soll aber keine neue Zeile geschrieben werden, sonderen vorher geprüft werden, ob diese ID in der data.txt vor kommt, wenn nicht? Dann Fehlermeldung wie bei:
if ($id <1 || $id>3) die("ID: [i]$id[/i] not used!");Ich möchte nicht immer diese Zeile neu anpassen, wenn ich einen download hinzufüge.
P.S. Das Script sollte ohne diese Zeile die ID prüfen.
so?
$dat = fopen($dbfile, "r");
$found = false;
if(!dat) die('[SYSTEMFEHLER]');
while(!feof($dat) {
$line = fgets($dat);
$line = explode('|', $line);
if($line[0] == $_GET['id']) $found = true;
}
if(!$found) die('ID exisitert nicht');Hi d4f,
Jaaaaaaaaaaaaaaaaa!
Es Funktioniert, vielen Dank für deine Mühe, habe es nun so gemacht:
include("session.inc.php");
$id = isset($_GET["id"]) ? $_GET["id"] : "";
if(!is_numeric($id) || $id>100) die("Hacking attempt!");
$dbfile = "data.txt";
$basedir = "/srv/www/htdocs/web/files/";
$dat = fopen($dbfile, "r");
$found = false;
while(!feof($dat)) {
$line = fgets($dat);
$line = explode("|", $line);
if($line[0] == $id) $found = true;
}
if(!$found) die("ID: [i]$id[/i] not used!");
fclose($dat);
$dat = fopen($dbfile, "r");
while($line = fgetcsv($dat, 1000, "|"))
{
if($line[0] == $id)
{
$switch = $line[1];
$digit = $line[2];
$digit++;
}
else
{
$erg[] = "$line[0]|$line[1]|$line[2]\n";
}
}
fclose($dat);
$dat = fopen($dbfile, "w");
fputs($dat, "$id|$switch|$digit\n");
$count = 0;
while($erg[$count] != "")
{
fputs($dat, $erg[$count]);
$count++;
}
fclose($dat);
$filename = sprintf("%s/%s", $basedir, $switch);
header("Content-Type: application/octet-stream");
$savename = basename($switch);
header("Content-Disposition: attachment; filename=\"$savename\"");
readfile($filename);Ein paar Fragen habe ich aber noch, muß denn 3-Mal fopen() in dem Script vorkommen? Oder lässt es sich mit einem fopen() und fclose() bewerkstelligen.
Und warum zuvor die Datei im Read Modus öffnen? Denn eigentlich sollte ja nur geschrieben werden (Downloads).
Warum kommt diese Meldung: "Notice: Undefined offset: 2" wenn das Script im Debug Modus läuft?
Codezeile, die die Meldung verursacht:
while($erg[$count] != "")also so dürfte es etwas resourcen so spaaren:
Da es hier um perfektion gehen soll, habe ich auch die änderungen gemacht, die auch nur ganz wenig bringen.
Z.b. die " gegen ' ersetzt. Da in den " auch nach variabelel ngeparst wird, was bei ' nicht getahen wird.
include('session.inc.php');
$id = @$_GET['id'];
if(!is_numeric($id) || $id>100) die('Hacking attempt!');
$dbfile = 'data.txt';
$basedir = '/srv/www/htdocs/web/files/';
$count = 0;
$found = false;
$fh = fopen($dbfile, 'w+');
foreach(file($dbfile) AS $line) //
{
if (strpos($line, $id.'|')===0) // spaart das explode. es wird einfach überprüft, ob die zeile mit id| anfängt. wgen dem === siehe http://de.php.net/manual/de/function.strpos.php stichwort typensicherheit
{
$found = true;
} else {
fwrite($fh, $line."\n");
$count ++;
}
}
if(!$found)
{
fclose($fh);
die('ID: [i]'.$id.'[/i] not used!');
}
if (!empty($id) && !empty($switch) && !empty($digit)) // Ich hoffe mal, du weist wo die 3 variabellen herkommen sollen ?!
fwrite($fh, "$id|$switch|$digit\n");
fclose($fh);
$filename = sprintf("%s/%s", $basedir, $switch);
header("Content-Type: application/octet-stream");
$savename = basename($switch);
header("Content-Disposition: attachment; filename=\"$savename\"");
readfile($filename);